noreply at github
Aug 19, 2009, 11:54 AM
Post #1 of 1
(539 views)
Permalink
|
|
[bricoleurs/bricolage] e1a49a: Refactor mark_as_published()
|
|
Branch: refs/heads/master
Home: http://github.com/bricoleurs/bricolage
Commit: e1a49add4cec047dcf585b232f1274fba3032d89
http://github.com/bricoleurs/bricolage/commit/e1a49add4cec047dcf585b232f1274fba3032d89
Author: David E. Wheeler <david[at]justatheory.com>
Date: 2009-08-19 (Wed, 19 Aug 2009)
Changed paths:
M lib/Bric/Biz/Asset/Business.pm
M lib/Bric/Changes.pod
M t/Bric/Biz/Asset/Business/DevTest.pm
Log Message:
-----------
Refactor mark_as_published()
I updated the docs for the new `mark_as_published()` method in
`Bric::Biz::Asset::Business`, removing some invalid notes, adding some valid
notes, and fixing some grammar nits. I also removed the call to `save()` at
the end of the method, because no method should call `save()`; the user should
always call it implicitly. I also changed the method so that it simply returns
if the document is already marked as published, as there is no point in
updating the publish date in this context. And finally, I added some tests
for the new method.
|
