When publishing, include the current record

README.md

@@ -7,9 +7,9 @@ Developed against ArchivesSpace v2.4.0 by Hudson Molonglo for Yale University.
 
 ## Summary
 
-Two new options are available in the `More` menu on the component toolbar for components that have children - `Publish All Children` and `Unpublish All Children`. When selected, all child components of the current component will be published or unpublished respectively.
+Two new options are available in the `More` menu on the component toolbar for components that have children - `Publish this and all children` and `Unpublish all children`. When selected, all child components of the current component will be published or unpublished respectively.
 
-The published state of the current component and its nested records is unchanged.
+In the case of publication the current record is also published. When unpublishing the published state of the current component and its nested records is unchanged.
 
 The published state of some `Note` records attached to child components is also unchanged.
 

backend/model/mixins/child_publisher.rb

@@ -9,7 +9,7 @@ def _publish_children!(setting, opts)
     object_graph.each do |model, ids|
       next unless model.publishable?
 
-      model.handle_publish_flag(model == self.class ? ids.reject {|i| i == self.id} : ids, setting)
+      model.handle_publish_flag(model == self.class && !setting ? ids.reject {|i| i == self.id} : ids, setting)
     end
   end
 

frontend/locales/en.yml

@@ -1,8 +1,8 @@
 en:
   plugins:
     child_publisher:
-      publish_toolbar_button: Publish All Children
-      unpublish_toolbar_button: Unpublish All Children
+      publish_toolbar_button: Publish this and all children
+      unpublish_toolbar_button: Unpublish all children
       messages:
-        published: Child records successfully published
+        published: This record and all its children successfully published
         unpublished: Child records successfully unpublished