Mailing List Archive

[bricoleurs/bricolage] 97e983: Add test for new `assets_count()` method.
Branch: refs/heads/master
Home: https://github.com/bricoleurs/bricolage

Commit: 97e9831fac8eb057a11099aae3ae031ffb6b815c
https://github.com/bricoleurs/bricolage/commit/97e9831fac8eb057a11099aae3ae031ffb6b815c
Author: David E. Wheeler <david@justatheory.com>
Date: 2011-08-05 (Fri, 05 Aug 2011)

Changed paths:
M t/Bric/Biz/Workflow/Parts/Desk/DevTest.pm

Log Message:
-----------
Add test for new `assets_count()` method.

Will be used to provide counts of assets on desks in the UI.


Commit: a0b62cee96c391e9672fed029d114c14752d6bee
https://github.com/bricoleurs/bricolage/commit/a0b62cee96c391e9672fed029d114c14752d6bee
Author: David E. Wheeler <david@justatheory.com>
Date: 2011-08-05 (Fri, 05 Aug 2011)

Changed paths:
M comp/widgets/wrappers/sideNav.mc
M lib/Bric/Biz/Workflow/Parts/Desk.pm

Log Message:
-----------
Add `assets_count()` to Desk and use it for desk counts.

This is a class method that returns a hash reference mapping desk IDs to counts of assets on those desks. Desks not in the hash refernce should be considered to have 0 assets.

Using it to provide desk counts when the "Show Desk Asset Counts" preference is on. This should be *far* more efficient than the old approach of loading every asset on every desk. [#239 state:resolved]


Compare: https://github.com/bricoleurs/bricolage/compare/e9584b5...a0b62ce