Changes between Initial Version and Version 1 of Ticket #17837, comment 18


Ignore:
Timestamp:
Apr 23, 2014, 10:36:45 AM (5 years ago)
Author:
Wouter Hager
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #17837, comment 18

    initial v1  
    11I'm sorry, but I disagree with this fix. I thought about it for a while, and I came to the conclusion that Observable is a piece of over-engineering that does a lot to obfuscate the workings of the ObjectStoreModel. In contrast with the JsonRest store, which is very concise, it requires additional code (SimpleQueryEngine) that repeats functionality that should have been taken care of server-side.
    22
    3 I vote for removing the Observable requirement from ObjectStoreModel and skim the code as much as possible.
     3I vote for removing the Observable requirement from ObjectStoreModel and skimming the code as much as possible.