[prev in list] [next in list] [prev in thread] [next in thread] 

List:       bricolage-commits
Subject:    [6083] Fix for when publish_status is not specified for
From:       theory () bricolage ! cc
Date:       2004-11-16 2:22:54
Message-ID: 20041116022254.20070.qmail () x1 ! develooper ! com
[Download RAW message or body]

Revision: 6083
Author:   theory
Date:     2004-11-15 18:22:53 -0800 (Mon, 15 Nov 2004)
ViewCVS:  http://viewsvn.bricolage.cc/?rev=6083&view=rev

Log Message:
-----------
Fix for when publish_status is not specified for SOAP::Workflow publish.

Modified Files:
--------------
    bricolage/branches/rev_1_8/lib/Bric/Changes.pod
    bricolage/branches/rev_1_8/lib/Bric/SOAP/Workflow.pm



["r6083-theory.diff" (r6083-theory.diff)]

Modified: bricolage/branches/rev_1_8/lib/Bric/Changes.pod
===================================================================
--- bricolage/branches/rev_1_8/lib/Bric/Changes.pod	2004-11-16 01:47:24 UTC (rev 6082)
+++ bricolage/branches/rev_1_8/lib/Bric/Changes.pod	2004-11-16 02:22:53 UTC (rev 6083)
@@ -32,6 +32,12 @@
 C<list()> methods of the story, media, and template classes is now properly
 supported by the SOAP C<list_ids> method, as well. [David]
 
+=item *
+
+Calls to publish documents via SOAP will no longer fail if the
+C<published_version> attribute is not specified and the document to be
+published has never been published before. [David]
+
 =back
 
 =head1 VERSION 1.8.3 (2004-11-09)

Modified: bricolage/branches/rev_1_8/lib/Bric/SOAP/Workflow.pm
===================================================================
--- bricolage/branches/rev_1_8/lib/Bric/SOAP/Workflow.pm	2004-11-16 01:47:24 UTC (rev 6082)
+++ bricolage/branches/rev_1_8/lib/Bric/SOAP/Workflow.pm	2004-11-16 02:22:53 UTC (rev 6083)
@@ -200,10 +200,11 @@
             $type = 'story';
             $obj  = Bric::Biz::Asset::Business::Story->lookup({
                 id => $id->value,
-                published_version => $published_only
+                ( $published_only ? ( published_version => 1) : ())
             });
 
-            throw_ap(error => "Unable to find story for story_id \"$id\".")
+            throw_ap(error => 'Unable to find story for story_id "'
+                              . $id->value . '".')
               unless $obj;
 
         } elsif ($id->name eq 'media_id') {


[prev in list] [next in list] [prev in thread] [next in thread]