# HG changeset patch
# User Chris Cannam
# Date 1315583781 -3600
# Node ID 202986dd17e4e02b3622b65cf38f358b67a393e8
# Parent 45ec06a8eec9b733a4b039c2f3c2f0684fe9d2b3# Parent 0ec052b8b297cf139b422fa62df2739dd12c1bda
Merge from branch "cannam_integration"
diff -r 45ec06a8eec9 -r 202986dd17e4 vendor/plugins/redmine_bibliography/app/controllers/publications_controller.rb
--- a/vendor/plugins/redmine_bibliography/app/controllers/publications_controller.rb Fri Sep 09 16:42:43 2011 +0100
+++ b/vendor/plugins/redmine_bibliography/app/controllers/publications_controller.rb Fri Sep 09 16:56:21 2011 +0100
@@ -31,6 +31,8 @@
@publication.projects << @project unless @project.nil?
if @publication.save
+ @publication.notify_authors_publication_added(@project)
+
flash[:notice] = "Successfully created publication."
redirect_to :action => :show, :id => @publication, :project_id => @project
else
diff -r 45ec06a8eec9 -r 202986dd17e4 vendor/plugins/redmine_bibliography/app/models/publication.rb
--- a/vendor/plugins/redmine_bibliography/app/models/publication.rb Fri Sep 09 16:42:43 2011 +0100
+++ b/vendor/plugins/redmine_bibliography/app/models/publication.rb Fri Sep 09 16:56:21 2011 +0100
@@ -17,7 +17,6 @@
has_and_belongs_to_many :projects, :uniq => true
before_save :set_initial_author_order
- after_save :notify_authors
# Ensure error message uses proper text instead of
# bibtex_entry.entry_type (#268). There has to be a better way to
@@ -30,14 +29,18 @@
end
end
- # Returns the mail adresses of users that should be notified
- def notify_authors
-
+ def notify_authors_publication_added(project)
self.authors.each do |author|
Rails.logger.debug { "Sending mail to \"#{self.title}\" publication authors." }
- Mailer.deliver_publication_added(author.user, self) unless author.user.nil?
+ Mailer.deliver_publication_added(author.user, self, project) unless author.user.nil?
end
-
+ end
+
+ def notify_authors_publication_updated(project)
+ self.authors.each do |author|
+ Rails.logger.debug { "Sending mail to \"#{self.title}\" publication authors." }
+ Mailer.deliver_publication_updated(author.user, self, project) unless author.user.nil?
+ end
end
diff -r 45ec06a8eec9 -r 202986dd17e4 vendor/plugins/redmine_bibliography/app/views/mailer/publication_added.text.html.rhtml
--- a/vendor/plugins/redmine_bibliography/app/views/mailer/publication_added.text.html.rhtml Fri Sep 09 16:42:43 2011 +0100
+++ b/vendor/plugins/redmine_bibliography/app/views/mailer/publication_added.text.html.rhtml Fri Sep 09 16:56:21 2011 +0100
@@ -1,3 +1,1 @@
-<%= link_to(h(@publication.title), @publication_url) %>
-
-<%= textilizable(@publication, :title, :only_path => false) %>
+<%= l(:mail_body_publication_added, :publication => @publication.title, :project => @project.name) %>
diff -r 45ec06a8eec9 -r 202986dd17e4 vendor/plugins/redmine_bibliography/app/views/mailer/publication_added.text.plain.rhtml
--- a/vendor/plugins/redmine_bibliography/app/views/mailer/publication_added.text.plain.rhtml Fri Sep 09 16:42:43 2011 +0100
+++ b/vendor/plugins/redmine_bibliography/app/views/mailer/publication_added.text.plain.rhtml Fri Sep 09 16:56:21 2011 +0100
@@ -1,3 +1,2 @@
-<%= @publication.title %>
-<%= @publication_url %>
+<%= l(:mail_body_publication_added, :publication => @publication.title, :project => @project.name) %>
diff -r 45ec06a8eec9 -r 202986dd17e4 vendor/plugins/redmine_bibliography/config/locales/en.yml
--- a/vendor/plugins/redmine_bibliography/config/locales/en.yml Fri Sep 09 16:42:43 2011 +0100
+++ b/vendor/plugins/redmine_bibliography/config/locales/en.yml Fri Sep 09 16:56:21 2011 +0100
@@ -137,6 +137,8 @@
label_author_19: Nineteenth author
label_author_20: Twentieth author
+ mail_subject_publication_added: "You have been added as an author to a new publication"
+ mail_body_publication_added: "A new publication (%{publication}) has been added to the project '%{project}.'"
diff -r 45ec06a8eec9 -r 202986dd17e4 vendor/plugins/redmine_bibliography/lib/bibliography/mailer_patch.rb
--- a/vendor/plugins/redmine_bibliography/lib/bibliography/mailer_patch.rb Fri Sep 09 16:42:43 2011 +0100
+++ b/vendor/plugins/redmine_bibliography/lib/bibliography/mailer_patch.rb Fri Sep 09 16:56:21 2011 +0100
@@ -10,16 +10,19 @@
# Example:
# publication_added(user) => tmail object
# Mailer.deliver_add_to_project(user) => sends an email to the registered user
- def publication_added(user, publication)
+ def publication_added(user, publication, project)
@publication = publication
+ @project = project
set_language_if_valid user.language
recipients user.mail
subject l(:mail_subject_publication_added, Setting.app_title)
body :publication_url => url_for( :controller => 'publications', :action => 'show', :id => publication.id ),
:publication_title => publication.title
+
render_multipart('publication_added', body)
+
end