noreply at github
Aug 5, 2011, 1:58 PM
Post #1 of 1
(217 views)
Permalink
|
|
[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 [at] justatheory>
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 [at] justatheory>
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
|
