Merge pull request ClosureTree#323 from Health-eFilings/fix-uninitialized-variable-warnings

Fix uninitialized variable warnings

Author: seuros

lib/closure_tree/hierarchy_maintenance.rb

@@ -20,7 +20,7 @@ def _ct_skip_sort_order_maintenance!
     end
 
     def _ct_validate
-      if !@_ct_skip_cycle_detection &&
+      if !(defined? @_ct_skip_cycle_detection) &&
         !new_record? && # don't validate for cycles if we're a new record
         changes[_ct.parent_column_name] && # don't validate for cycles if we didn't change our parent
         parent.present? && # don't validate if we're root
@@ -64,7 +64,7 @@ def _ct_before_destroy
 
     def rebuild!(called_by_rebuild = false)
       _ct.with_advisory_lock do
-        delete_hierarchy_references unless @was_new_record
+        delete_hierarchy_references unless (defined? @was_new_record) && @was_new_record
         hierarchy_class.create!(:ancestor => self, :descendant => self, :generations => 0)
         unless root?
           _ct.connection.execute <<-SQL.strip_heredoc