tweaked destroy and save to be a little less brittle

git-svn-id: http://svn.addictedtonew.com/public/gems/mirrored@176 fe7eae16-9a24-0410-a59d-9e59979e88be

History.txt

@@ -1,3 +1,6 @@
+0.1.5
+* tweaked destroy and save to be a little less brittle
+
 0.1.4
 * screwed up 0.1.3, this correctly fixes hpricot issue
 

License.txt

@@ -1,20 +1,19 @@
 Copyright (c) 2007 John Nunemaker
 
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-"Software"), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
 
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
 
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
-NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
-LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
-OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
-WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.

Manifest.txt

@@ -20,12 +20,6 @@ setup.rb
 tasks/deployment.rake
 tasks/environment.rake
 tasks/website.rake
-test/test_base.rb
-test/test_date.rb
-test/test_helper.rb
-test/test_post.rb
-test/test_tag.rb
-test/test_update.rb
 test/fixtures/xml/add_post_done.xml
 test/fixtures/xml/all_posts.xml
 test/fixtures/xml/all_posts_by_tag.xml
@@ -42,6 +36,11 @@ test/fixtures/xml/recent_posts_by_tag.xml
 test/fixtures/xml/recent_posts_by_tag_and_count.xml
 test/fixtures/xml/tag_rename.xml
 test/fixtures/xml/tags.xml
+test/test_base.rb
+test/test_date.rb
+test/test_helper.rb
+test/test_post.rb
+test/test_tag.rb
+test/test_update.rb
 website/css/common.css
-website/images
-website/index.html
+website/index.html

lib/mirrored/post.rb

@@ -65,8 +65,12 @@ def find(*args)
       #   Mirrored::Post.delete('http://microsoft.com')   # => aliased version
       def destroy(url)
         doc = Hpricot::XML(connection.get('posts/delete', :url => url))
-        code = doc.at('result')['code']
-        code == 'done' ? true : false
+        if doc && doc.at('result')
+          code = doc.at('result')['code']
+          code == 'done' ? true : false
+        else
+          false
+        end
       end
       alias :delete :destroy
     end
@@ -85,8 +89,12 @@ def save(replace=false)
       attrs = to_h.merge((replace) ? {:replace => 'yes'} : {:replace => 'no'})
       puts attrs.inspect
       doc = Hpricot::XML(self.class.connection.get('posts/add', attrs))
-      @code = doc.at('result')['code']
-      (@code == 'done') ? true : false
+      if doc && doc.at('result')
+        @code = doc.at('result')['code']
+        (@code == 'done') ? true : false
+      else
+        false
+      end
     end
 
     def to_h #:nodoc:

lib/mirrored/version.rb

@@ -2,7 +2,7 @@ module Mirrored #:nodoc:
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 1
-    TINY  = 4
+    TINY  = 5
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end