Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #40748, comment 5


Ignore:
Timestamp:
05/14/2017 08:12:39 PM (9 years ago)
Author:
azaozz
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #40748, comment 5

    initial v1  
    1010It is meant to do only one thing: provide a small amount of data for the dashboard widget.
    1111
    12 On top of that I think this conversion sends out the wrong message. No matter how I try to look at it, the example is negative. It replaces '''24 lines''' of simple, straightforward, easy to read and understand code with exactly '''498 lines''' of much more complex, harder to read and understand code contained in two new files! In addition the new code seems to introduce bugs, edge cases and unneeded functionality. How can we expect plugin authors to want to do something like that in their plugins?
     12On top of that I think this conversion sends out the wrong message. No matter how I try to look at it, the example is negative. It replaces '''24 lines''' of simple, straightforward, easy to read and understand code with exactly '''498 lines''' of much more complex, harder to read and understand code contained in two new files! In addition the new code seems to introduce bugs, edge cases and unneeded functionality. Is that really a typical example on how to use the REST API instead of admin-ajax? How can we expect plugin authors to want to do something like that in their plugins?
    1313