{"push_id":1023509460,"size":1,"distinct_size":1,"ref":"refs/heads/5.0","head":"bc2411b02fa36dd6b36e5e62ecd43fa05d5f952b","before":"710f42f5733cd512942d19f420bae8a52529286b","commits":[{"sha":"bc2411b02fa36dd6b36e5e62ecd43fa05d5f952b","author":{"email":"719ea5a00d36414593f51e42d6b2858635f39069@gmail.com","name":"thet"},"message":"[fc] Repository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:51:04+01:00\nAuthor: Tommy Yu (metatoaster) <tommy.yu@auckland.ac.nz>\nCommit: https://github.com/plone/mockup/commit/50d4b0d937306bf014bb6835ed3c861ab3342509\n\nBasic refactoring/cleanup for configurability.\n\n- New base test description for structure\n- This one will contain customization and have no data (to verify some\n  basic implementation details).\n- Changed setup for first test to be JSON format for better syntax\n  visualization.\n- Empty body.html added to afterEach for cleanup (test teardown).\n\nPaste button no longer required to exist.\n\n- Ensure that a missing paste button will prevent the table from being\n  rendered.\n\nAbility to customize the cookie for columns\n\n- Define a new key \"activeColumnsCookie\" for structure.\n- This enables view specific column settings (i.e. a view on a list of\n  users may have sufficiently different columns such that the settings\n  for folder_contents shouldn't apply or interfere with at all).\n- Needed adjustments in the view and template for tablerow to prevent\n  rendering from being aborted to missing data.\n- Still need a bare minimum set of attributes being available (i.e.\n  Title and getURL).\n\nFormalize definition of nested params in structure\n\n- Provide default parameters with the _default_ prefix.\n- Replace the demoButton with _default_buttons to be consistent.\n- Provide a generalized helper function for the replacement of the\n  object/array type options for the structure pattern.\n- First usage on this is to write more tests, in this case document that\n  the buttons can be completely replaced, or even completely removed.\n\nFiles changed:\nM mockup/patterns/structure/js/views/app.js\nM mockup/patterns/structure/js/views/columns.js\nM mockup/patterns/structure/js/views/tablerow.js\nM mockup/patterns/structure/pattern.js\nM mockup/patterns/structure/templates/tablerow.xml\nM mockup/tests/pattern-structure-test.js\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:51:04+01:00\nAuthor: Tommy Yu (metatoaster) <tommy.yu@auckland.ac.nz>\nCommit: https://github.com/plone/mockup/commit/89de866dff89a455bd4102c84a3fa8f9a0bcc34b\n\nMajor ActionMenu refactoring for extensibility.\n\nTest for per item dropdown action menu click\n\n- Ensure that at least one of them works (move-top).\n- See that the defaults will continue to work when changes to migrate\n  generation of the items and bindings are moved to a more configurable\n  definition.\n- Make it actually do the full round trip including the existing button.\n\nInitial split of Actions from ActionMenu\n\n- Per-item action methods should not be defined directly within the view\n  class; view class should only deal with rendering/registration of the\n  events, and those events should be registered within a seaparate model\n  which this view class should make use of\n- Baby steps: want to make it possible to hook in events that could be\n  defined by other libraries.\n\nHooking in the event with requirejs\n\n- Provide a way to specify dynamically where the modules are for a given\n  event along with the label.\n\nConfigurable menu options.\n\n- Move option visibility logic out of template\n- Instead, construct the table for this shared between the event method\n  and the template.\n- Enable the user-specifiable option for the template.\n- Include a test for the customization of action menu.\n- Pass down menu option into current folder actions\n\nHave action menu tests define/use custom modules\n\n- Show that the internal procedure of requiring/using modules loaded via\n  requirejs functions as expected.\n\nHook moveUrl to the availability of rearrangement\n\n- Don't apply the rearrange class to the parent of the table if that was\n  not provided in the constructor.\n- Also don't provide the move-top and move-bottom actions for that case.\n\nAbility to specify action at the table level\n\n- Add in tests to ensure correctness of existing work.\n- Still need to figure out how to do this at the per item level.\n\nVarious per item actionmenu tests\n\n- Fixed the missing entries when used (have to clone that object because\n  javascript).\n- Actually add in the tests for the functionalities for those actionmenu\n  items to ensure that they are triggered to ensure they stay correct.\n- Verify certain xhr requests.\n\nSplit up the navigation stuff\n\n- getWindow now a utility function - make it stubbable because assigning\n  window.location will result in test framework triggering a disconnect.\n- Have the new module that only deal with general navigation such as\n  opening a link.\n\nUser provided link click handlers\n\n- The actions for handling folder and standard item clicks can be\n  defined similar to how actionmenu was done.\n- Removed all the extraneous methods that were duplicated for this in\n  the previous commit(s).\n\nDo a proper integration test for actionmenu.\n\n- Show that it works from within a table also.\n\nSplit up ActionMenu into view and menu component\n\n- Renamed existing ActionMenu to ActionMenuView (maintaining consistency\n  with other js files).\n- Split out the default ActionMenu generation functionality into its own\n  module.\n\nAssign menuOptions iff menuGenerator return object\n\n- Prevent some basic configuration failure cases - at least show data\n  even if action menu might be missing.\n\nFinalize menuGenerator integration\n\n- This allow the generator to be specified at the app level\n- Test demonstrates how menu can look into each model for given row to\n  generate the appropriate actions.\n\nCheck that unselect actually unselect stuff\n\n- Find out that if the raw data is missing the id attribute for a given\n  item, the de-selection will not be able to unselect the checkbox.\n\nTesting selectAll within a folder in table\n\n- Show that this functionality also work, but the intial test data had\n  to be manipulated\n- Also had to reset the queryHelper, because the currentPath is not\n  properly cleansed between tests.\n- Tests that relied on the above had to be adjusted to go back down to\n  the expected currentPath.\n\nFiles changed:\nA mockup/patterns/structure/js/actionmenu.js\nA mockup/patterns/structure/js/actions.js\nA mockup/patterns/structure/js/navigation.js\nM mockup/js/utils.js\nM mockup/patterns/structure/js/collections/result.js\nM mockup/patterns/structure/js/views/actionmenu.js\nM mockup/patterns/structure/js/views/app.js\nM mockup/patterns/structure/js/views/table.js\nM mockup/patterns/structure/js/views/tablerow.js\nM mockup/patterns/structure/pattern.js\nM mockup/patterns/structure/templates/actionmenu.xml\nM mockup/tests/pattern-structure-test.js\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:51:04+01:00\nAuthor: Tommy Yu (metatoaster) <tommy.yu@auckland.ac.nz>\nCommit: https://github.com/plone/mockup/commit/f0cd4dd231fb95583cfb63043ba7204c2f74a380\n\nDetangle object relationships, formalize path api\n\n- This was done to clarify the scope and hierarchy of the affected\n  objects.\n\n- The AppView instance is where the path setting should be done, even\n  though it ultimately pokes into the methods by the same name within\n  ResultCollection.\n\n  - Code that previously used queryHelper.currentPath now delegate this\n    through the methods provided by AppView.\n\n- Formalizing what needs to be implemented for custom ResultCollection\n  models - it needs to provide getCurrentPath/setCurrentPath along with\n  the relevant Backbone.Paginator.requestPager methods.  This should be\n  documented.\n\n  - The instance of AppView.collection now keeps track of the path that\n    the AppView should be on - no longer requiring fetching this value\n    from queryHelper\n\n- A completely custom ResultCollection can be now provided.  It does not\n  need to follow the existing usage of the QueryHelper/Parser class as\n  custom implementations don't need to deal with the Plone Catalog.\n\n- Default ResultCollection implementation now constructs queryParser and\n  queryHelper inside itself to keep as much of the Plone catalog and\n  content type specific information out of the pattern itself to aid in\n  allowing reuse without the unwanted Plone bits (specifically with the\n  queryHelper stuff, it really had no place on the top level option\n  object).\n\n- Default implementation of the queryParser and construction of\n  queryHelper should remain unchanged except for its references to\n  certain attributes that exist on the parent app objects (such as the\n  AppView.toolbar).\n\nFormalize the previous changes with a final test.\n\n- Created a test that cover the case where the QueryHelper (and\n  collection.queryParser) is triggered, such that they together should\n  generate the correct query parameters for consumption by the\n  vocabulary endpoint within a Plone instance.\n\n- The test also serves to finalize the clarified version of the object\n  relationships between the structure pattern, AppView, ResultCollection\n  and QueryHelper.  The previously tangled web of relationships\n  (specifically the setting/access of the current path to be\n  accessed/represented by the structure view) is more controlled.\n\n- To really hammer in this point, AppView has an instance of\n  ResultCollection, and it gained complete ownership of QueryHelper as\n  it utlimately controls what gets populated inside the table, despite\n  how QueryHelper needs to have access to the certain attributes of the\n  pattern itself.\n\nEnsure that the selectAll option work still\n\n- Moving the queryHelper into the collection broke that, turns out this\n  wasn't tested before.\n\nFiles changed:\nM mockup/patterns/structure/js/actions.js\nM mockup/patterns/structure/js/collections/result.js\nM mockup/patterns/structure/js/navigation.js\nM mockup/patterns/structure/js/views/app.js\nM mockup/patterns/structure/js/views/table.js\nM mockup/patterns/structure/js/views/upload.js\nM mockup/patterns/structure/pattern.js\nM mockup/tests/pattern-structure-test.js\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:51:04+01:00\nAuthor: Tommy Yu (metatoaster) <tommy.yu@auckland.ac.nz>\nCommit: https://github.com/plone/mockup/commit/743e2f26a35aa2bd6d8cf2b308fb18e5627adf19\n\nHave a dedicated variable for the view url\n\n- For the link that opens a new window, there will be cases where the\n  view suffix is not desirable.  This allows the data source to provide\n  an explicit one.  Fallback behavior is what it was before.\n\nFiles changed:\nM mockup/patterns/structure/js/views/tablerow.js\nM mockup/patterns/structure/templates/tablerow.xml\nM mockup/tests/pattern-structure-test.js\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:51:05+01:00\nAuthor: Tommy Yu (metatoaster) <tommy.yu@auckland.ac.nz>\nCommit: https://github.com/plone/mockup/commit/0238ccddea8aa7e21f105da8ef68ebe4177ecaba\n\nSupport presenting IPublishTraverse view paths\n\n- This change ensures that it is possible to trigger the events off the\n  correct base url as the views will not have the actual real views that\n  would have been resolved by the event.\n- Ensure that the push/popstates are properly handled/triggered.\n\nAlso more tests for window pushState\n\n- Added the configuration needed to trigger that.\n- Added the required window mocks to allow the pushState to be tested.\n- Ensure that the URL argument passed is as expected.\n\nFiles changed:\nM mockup/patterns/structure/js/views/app.js\nM mockup/tests/pattern-structure-test.js\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:51:05+01:00\nAuthor: Tommy Yu (metatoaster) <tommy.yu@auckland.ac.nz>\nCommit: https://github.com/plone/mockup/commit/43b236d8f9a44704c72cb2277218af5da6cd18f2\n\nStructure: pushState url format parameter\n\n- Provide an alternative way to set the pushState url, using a format\n  string that make use of the {path} token for substitution as an\n  alternative to the somewhat more opaque urlStructure parameter.\n\nFiles changed:\nM mockup/patterns/structure/js/views/app.js\nM mockup/tests/pattern-structure-test.js\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:51:05+01:00\nAuthor: Tommy Yu (metatoaster) <tommy.yu@auckland.ac.nz>\nCommit: https://github.com/plone/mockup/commit/ac894ee30d41618175a69de6a4c06b7ca64372dc\n\nChangelog for what's been done to structure.\n\nFiles changed:\nM CHANGES.rst\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:53:04+01:00\nAuthor: Johannes Raggam (thet) <thetetet@gmail.com>\nCommit: https://github.com/plone/mockup/commit/4fa0384adbf38f3b3af09916b0977990cf1f63cf\n\nraise version\n\nFiles changed:\nM CHANGES.rst\nM package.json\nRepository: mockup\nBranch: refs/heads/master\nDate: 2016-03-16T15:53:39+01:00\nAuthor: Johannes Raggam (thet) <thetetet@gmail.com>\nCommit: https://github.com/plone/mockup/commit/ba76b86e6a665e60ff12ae34321efd2833c5b069\n\nMerge pull request #618 from plone/metatoaster-structure-master\n\nWIP: Making structure pattern more flexible to customization options\n\nFiles changed:\nA mockup/patterns/structure/js/actionmenu.js\nA mockup/patterns/structure/js/actions.js\nA mockup/patterns/structure/js/navigation.js\nM CHANGES.rst\nM mockup/js/utils.js\nM mockup/patterns/structure/js/collections/result.js\nM mockup/patterns/structure/js/views/actionmenu.js\nM mockup/patterns/structure/js/views/app.js\nM mockup/patterns/structure/js/views/columns.js\nM mockup/patterns/structure/js/views/table.js\nM mockup/patterns/structure/js/views/tablerow.js\nM mockup/patterns/structure/js/views/upload.js\nM mockup/patterns/structure/pattern.js\nM mockup/patterns/structure/templates/actionmenu.xml\nM mockup/patterns/structure/templates/tablerow.xml\nM mockup/tests/pattern-structure-test.js\nM package.json","distinct":true,"url":"https://api.github.com/repos/plone/buildout.coredev/commits/bc2411b02fa36dd6b36e5e62ecd43fa05d5f952b"}]}