Description:
Last change round of charm store data structures
This change is hopefully the last round of changes, at least for a long
while, to the underlying charm store infrastructure. It is more deletes
than additions, and changes the code to take advantage of the changes
Kapil made to the charm store.
I made some decisions as to how to factor some of these things and would
be happy to describe my rationale for changes if desired. I'm thinking
particularly of the immediate creation of charm objects as search
results. That allowed for some simplifications and I think is mostly a
win.
Affected files:
A [revision details]
M app/app.js
M app/models/charm.js
M app/modules.js
M app/store/charm.js
M app/templates/charm-search-result.handlebars
M app/views/charm-search.js
M test/data/search_results.json
M test/data/series_search_results.json
M test/test_app.js
M test/test_charm_configuration.js
M test/test_charm_search.js
M test/test_charm_store.js
M test/test_model.js
M test/test_service_view.js
Reviewers: mp+131086_ code.launchpad. net,
Message:
Please take a look.
Description:
Last change round of charm store data structures
This change is hopefully the last round of changes, at least for a long
while, to the underlying charm store infrastructure. It is more deletes
than additions, and changes the code to take advantage of the changes
Kapil made to the charm store.
I made some decisions as to how to factor some of these things and would
be happy to describe my rationale for changes if desired. I'm thinking
particularly of the immediate creation of charm objects as search
results. That allowed for some simplifications and I think is mostly a
win.
The sorting code is simplified yet again.
Thanks
Gary
https:/ /code.launchpad .net/~gary/ juju-gui/ simplifycharmst ore/+merge/ 131086
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/6733067/
Affected files: charm-search- result. handlebars charm-search. js search_ results. json series_ search_ results. json charm_configura tion.js charm_search. js charm_store. js service_ view.js
A [revision details]
M app/app.js
M app/models/charm.js
M app/modules.js
M app/store/charm.js
M app/templates/
M app/views/
M test/data/
M test/data/
M test/test_app.js
M test/test_
M test/test_
M test/test_
M test/test_model.js
M test/test_