[gallery/gallery3] c13284: Change flow so that we assert after the line that ...

GitHub <noreply-9UaJU3cA/F/[email protected]> Mon, 29 Apr 2013 15:34:19 -0700
Newsgroups gmane.comp.web.gallery.cvs
Message-ID <517ef56b4a20d_55005cbe1c91567@hookshot-fe2-pe1-prd.aws.github.net.mail>
  Branch: refs/heads/kohana_3
  Home:   https://github.com/gallery/gallery3
  Commit: c13284b9d3470a967e64325e8624a290ca2c713c
      https://github.com/gallery/gallery3/commit/c13284b9d3470a967e64325e8624a290ca2c713c
  Author: Bharat Mediratta <[email protected]>
  Date:   2013-04-28 (Sun, 28 Apr 2013)

  Changed paths:
    M modules/gallery/tests/Item_Rest_Helper_Test.php

  Log Message:
  -----------
  Change flow so that we assert after the line that should fail.  Avoid returns.


  Commit: 558cb725d83a4cbbe622838b7d17b05a9c5f9718
      https://github.com/gallery/gallery3/commit/558cb725d83a4cbbe622838b7d17b05a9c5f9718
  Author: Bharat Mediratta <[email protected]>
  Date:   2013-04-29 (Mon, 29 Apr 2013)

  Changed paths:
    M modules/g2_import/classes/G2Import/G2Import.php
    M modules/gallery/classes/Gallery/Access.php
    M modules/gallery/classes/Gallery/Breadcrumb.php
    M modules/gallery/classes/Gallery/Controller/Albums.php
    M modules/gallery/classes/Gallery/Controller/Permissions.php
    M modules/gallery/classes/Gallery/Controller/Quick.php
    M modules/gallery/classes/Gallery/Controller/Uploader.php
    M modules/gallery/classes/Gallery/Hook/GalleryEvent.php
    M modules/gallery/classes/Gallery/Hook/GalleryRss.php
    M modules/gallery/classes/Gallery/Hook/Rest/Items.php
    M modules/gallery/classes/Gallery/Hook/Rest/Tree.php
    M modules/gallery/classes/Gallery/Item.php
    M modules/gallery/classes/Gallery/Model/Item.php
    M modules/gallery/classes/Gallery/ORM.php
    M modules/gallery/classes/Gallery/ORM/MPTT.php
    M modules/gallery/classes/Gallery/View/Theme.php
    M modules/gallery/orphans/views/form_uploadify.html.php
    M modules/gallery/tests/Items_Rest_Helper_Test.php
    M modules/gallery/tests/ORM_MPTT_Test.php
    M modules/notification/classes/Notification/Notification.php
    M modules/notification/views/notification/item_deleted.html.php
    M modules/organize/classes/Organize/Controller/Organize.php
    M modules/organize/classes/Organize/Hook/OrganizeEvent.php
    M modules/rss/classes/Rss/Hook/RssTheme.php
    M modules/server_add/classes/ServerAdd/Hook/ServerAddEvent.php
    M modules/server_add/views/server_add/tree_dialog.html.php
    M modules/slideshow/classes/Slideshow/Hook/SlideshowEvent.php

  Log Message:
  -----------
  Reimplement ORM_MPTT relationships with a similar API to regular ORM
relationships per a suggestion from Shad[1].

Exmaple before/after pairs:

   $item->children()
-->$item->children->find_all()

   $item->viewable()->children()
-->$item->children->viewable()->find_all()

   $item->children(1, 2, array("type", "=", "album"))
-->$item->children->where("type", "=", "album")->limit(1)->offset(2)->find_all()

Consequences:
- Simpler, more consistent API for dealing with relationships
- Get rid of ORM::unloaded_instance() hack
- Reduced code size
- Longer call chains where it's used

[1] https://github.com/gallery/gallery3/commit/88b6865f472b4203b9ef04d4b99e3310ecb960d2


  Commit: 177af351620337e05a6e0b845981d3237affa68f
      https://github.com/gallery/gallery3/commit/177af351620337e05a6e0b845981d3237affa68f
  Author: Bharat Mediratta <[email protected]>
  Date:   2013-04-29 (Mon, 29 Apr 2013)

  Changed paths:
    M modules/gallery/classes/Gallery/Model/Item.php

  Log Message:
  -----------
  Add a todo:
      // @TODO: revisit this - it's silly to have a design which allows us to not have an identity
      // provider.


  Commit: f8d56a78e66c99bb85e651d346a87090a3249718
      https://github.com/gallery/gallery3/commit/f8d56a78e66c99bb85e651d346a87090a3249718
  Author: Bharat Mediratta <[email protected]>
  Date:   2013-04-29 (Mon, 29 Apr 2013)

  Changed paths:
    M modules/gallery/classes/Gallery/Model/Item.php

  Log Message:
  -----------
  Fix indentation.


  Commit: 1309782d05b9064974f83ab06b955c5bf882a9c2
      https://github.com/gallery/gallery3/commit/1309782d05b9064974f83ab06b955c5bf882a9c2
  Author: Bharat Mediratta <[email protected]>
  Date:   2013-04-29 (Mon, 29 Apr 2013)

  Changed paths:
    M modules/gallery/classes/Gallery/Access.php
    M modules/gallery/classes/Gallery/Hook/GalleryEvent.php
    M modules/gallery/classes/Gallery/Model/Item.php
    M modules/gallery/classes/Gallery/ORM/MPTT.php
    M modules/gallery/tests/Access_Test.php

  Log Message:
  -----------
  Make ORM_MPTT::parent a regular ORM relationship.  This exposes issues
where relationships can get stale and occasionally need to be reloaded
- Access_Test::test_moved_items_inherit_new_permissions started
failing and highlighted it.  So reload some relationships if we
believe that they might be out of date.  This wasn't necessary before
we started making more things relationships (and possibly was one of
the original reasons why we didn't use relationships for this code).


  Commit: acced96c316c3ae73f5caf0fd582b36a3877f21d
      https://github.com/gallery/gallery3/commit/acced96c316c3ae73f5caf0fd582b36a3877f21d
  Author: Bharat Mediratta <[email protected]>
  Date:   2013-04-29 (Mon, 29 Apr 2013)

  Changed paths:
    R modules/formo/views/formo/field_template.php
    R modules/formo/views/formo/form_template.php
    R modules/formo/views/formo/group_template.php
    R modules/formo/views/formo/opts/checkboxes_template.php
    R modules/formo/views/formo/opts/radios_template.php
    R modules/formo/views/formo/opts/select_template.php
    R modules/formo/views/formo_bootstrap/checkbox_template.php
    R modules/formo/views/formo_bootstrap/field_template.php
    R modules/formo/views/formo_bootstrap/form_template.php
    R modules/formo/views/formo_bootstrap/group_template.php
    R modules/formo/views/formo_bootstrap/opts/checkboxes_template.php
    R modules/formo/views/formo_bootstrap/opts/radios_template.php
    R modules/formo/views/formo_bootstrap/opts/select_template.php
    M modules/gallery/classes/Gallery/Formo.php
    M modules/gallery/config/formo.php
    A modules/gallery/views/formo/field_template.php
    A modules/gallery/views/formo/form_template.php
    A modules/gallery/views/formo/group_template.php
    A modules/gallery/views/formo/opts/checkboxes_template.php
    A modules/gallery/views/formo/opts/radios_template.php
    A modules/gallery/views/formo/opts/select_template.php
    M themes/wind/assets/screen-fix-ie.css

  Log Message:
  -----------
  Merge branch 'kohana_3' of github.com:gallery/gallery3 into kohana_3


Compare: https://github.com/gallery/gallery3/compare/d49fca7d909a...acced96c316c

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr

__[ g a l l e r y - c h e c k i n s ]_________________________

[ list info/archive --> http://gallery.sf.net/lists.php ]
[ gallery info/FAQ/download --> http://gallery.sf.net ]