app/views/mailer/wiki_add.text.html.rhtml app/views/mailer/wiki_add.text.plain.rhtml app/views/mailer/wiki_edit.text.html.rhtml app/views/mailer/wiki_edit.text.plain.rhtml app/views/mailer/wiki_remove.text.html.rhtml app/views/mailer/wiki_remove.text.plain.rhtml @@ -50,7 +50,7 @@ class AttachmentsController < ApplicationController def to_notify? if @attachment.container.class.respond_to?(:notify?) - @attachment.container.class.notify?(:update) + @attachment.container.class.notify?(:updated) else Setting.notified_events.include?(NotificationKeys::FILE_ADDED) end app/controllers/attachments_controller.rb @@ -80,16 +80,26 @@ class WikiController < ApplicationController #@content.comments = params[:content][:comments] @content.attributes = params[:content] @content.author = User.current + # if page is new @page.save will also save content, but not if page isn't a new record - if (@page.new_record? ? @page.save : @content.save) - redirect_to page_link + if @page.new_record? + result = @page.save + if result && WikiPage.notify? + Mailer.deliver_wiki_add(@content) + end + else + result = @content.save + if result && WikiPage.notify? + Mailer.deliver_wiki_edit(@content) + end end + redirect_to page_link if result end rescue ActiveRecord::StaleObjectError # Optimistic locking exception flash[:error] = l(:notice_locking_conflict) end - + # rename a page def rename return render_403 unless editable? @@ -135,6 +145,7 @@ class WikiController < ApplicationController def destroy return render_403 unless editable? @page.destroy + Mailer.deliver_wiki_remove(@page, User.current) if WikiPage.notify? redirect_to :action => 'special', :id => @project, :page => 'Page_index' end @@ -178,8 +189,7 @@ class WikiController < ApplicationController def add_attachment return render_403 unless editable? attachments = attach_files(@page, params[:attachments]) - if !attachments.empty? && - Setting.notified_events.include?(NotificationKeys::WIKI_EDIT) + if !attachments.empty? && WikiPage.notify? Mailer.deliver_attachments_added(attachments) end app/controllers/wiki_controller.rb @@ -35,7 +35,7 @@ class Document < ActiveRecord::Base end end - def self.notify?(action) + def self.notify?(action = nil) Setting.notified_events.include?(NotificationKeys::DOCUMENT_ADDED) end end app/models/document.rb @@ -284,7 +284,7 @@ class Issue < ActiveRecord::Base end def self.notify?(action) - Setting.notified_events.include?(NotificationKeys.const_get('ISSUE_%sD' % action.to_s.upcase)) + Setting.notified_events.include?(NotificationKeys.const_get('ISSUE_%s' % action.to_s.upcase)) end private app/models/issue.rb @@ -69,11 +69,41 @@ class Mailer < ActionMailer::Base def document_added(document) redmine_headers 'Project' => document.project.identifier recipients document.project.recipients - subject "[#{document.project.name}] #{l(:label_document_new)}: #{document.title}" + subject "[#{document.project.name}] #{l(:label_document_added)}: #{document.title}" body :document => document, :document_url => url_for(:controller => 'documents', :action => 'show', :id => document) end + def wiki_add(content) + page = content.page + redmine_headers 'Project' => page.project.identifier + recipients page.project.recipients + subject "[#{page.project.name}] #{l(:label_wiki_added)}: #{page.title}" + body :page => page, :content => content, + :page_url => url_for({:controller => 'wiki'}.merge(WikiController.page_link(page))) + end + + def wiki_remove(page, actor) + redmine_headers 'Project' => page.project.identifier + recipients page.project.recipients + subject "[#{page.project.name}] #{l(:label_wiki_deleted)}: #{page.title}" + body :page => page, :actor => actor, :pages_index_url => + url_for(:controller => 'wiki', :action => 'special', :id => page.project, + :page => 'Page_index') + end + + def wiki_edit(content) + page = content.page + redmine_headers 'Project' => page.project.identifier + recipients page.project.recipients + subject "[#{page.project.name}] #{l(:label_wiki_changed)}: #{page.title}" + body :page => page, :content => content, + :page_url => url_for({:controller => 'wiki'}. + merge(WikiController.page_link(page))), + :change_url => url_for(:controller => 'wiki', :action => 'diff', + :page => page.title, :version => content.version, :id => page.project) + end + def attachments_added(attachments) attachments_action(attachments) do |container| subject "[#{container.project.name}] #{l(:label_attachment_new)}" app/models/mailer.rb @@ -126,7 +126,7 @@ class WikiPage < ActiveRecord::Base self.parent = parent_page end - def self.notify?(action) + def self.notify?(action = nil) Setting.notified_events.include?(NotificationKeys::WIKI_EDIT) end app/models/wiki_page.rb @@ -311,6 +311,11 @@ label_document: Document label_document_new: New document label_document_plural: Documents label_document_added: Document added +label_wiki_added: Wiki page added +label_wiki_changed: Wiki page changed +label_wiki_changed_word: changed +label_wiki_deleted: Wiki page was deleted +label_wiki_deleted_by: Wiki page '%s' was deleted by %s label_role: Role label_role_plural: Roles label_role_new: New role @@ -680,6 +685,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess text_issue_added: Issue %s has been reported by %s. text_issue_updated: Issue %s has been updated by %s. text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ? +text_wiki_page_added: Wiki page has been added by %s +text_wiki_page_changed: Wiki page has been %s by %s +text_with_comment: with comment '%s' text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ? text_issue_category_destroy_assignments: Remove category assignments text_issue_category_reassign_to: Reassign issues to this category lang/en.yml @@ -579,6 +579,11 @@ label_wiki_edit: Редактирование Wiki label_wiki_edit_plural: Редактирования Wiki label_wiki_page: Страница Wiki label_wiki_page_plural: Страницы Wiki +label_wiki_added: Добавлена страница Wiki +label_wiki_changed: Изменена страница Wiki +label_wiki_changed_word: изменена +label_wiki_deleted: Удалена страница Wiki +label_wiki_deleted_by: Страница Wiki '%s' была удалена пользователем %s label_workflow: Последовательность действий label_year: Год label_yesterday: вчера @@ -727,16 +732,13 @@ text_assign_time_entries_to_project: Прикрепить зарегистрир text_caracters_maximum: %d символов(а) максимум. text_caracters_maximum: Максимум %d символов(а). text_caracters_minimum: Должно быть не менее %d символов. -text_caracters_minimum: Должно быть не менее %d знаков. text_clear_to_recalculate_time_by_range: Очистите, чтобы пересчитать по времени окончания text_comma_separated: Допустимы несколько значений (разделенные запятой). -text_comma_separated: Допустимы несколько значений (через запятую). text_custom_field_possible_values_info: 'One line for each value' text_default_administrator_account_changed: Учетная запись администратора по умолчанию изменена text_destroy_time_entries: Удалить зарегистрированное время text_destroy_time_entries_question: Вы собираетесь удалить %.02f часа(ов) прикрепленных за этой задачей. text_diff_truncated: '... Этот diff ограничен, так как превышает максимальный отображаемый размер.' -text_email_delivery_not_configured: "Параметры работы с почтовым сервером не настроены и функция уведомления по email не активна.\nНастроить параметры для вашего SMTP сервера вы можете в файле config/email.yml. Для применения изменений перезапустите приложение." text_email_delivery_not_configured: "Параметры работы с почтовым сервером не настроены и функция уведомления по email не активна.\nНастроить параметры для Вашего SMTP-сервера Вы можете в файле config/email.yml. Для применения изменений перезапустите приложение." text_enumeration_category_reassign_to: 'Назначить им следующее значение:' text_enumeration_destroy_question: '%d объект(а,ов) связаны с этим значением.' @@ -745,7 +747,6 @@ text_hours_short: ч text_in_progress: в процессе text_issue_added: По задаче %s был создан отчет (%s). text_issue_category_destroy_assignments: Удалить назначения категории -text_issue_category_destroy_question: Несколько задач (%d) назначено в данную категорию. Что вы хотите предпринять? text_issue_category_destroy_question: Несколько задач (%d) назначено в данную категорию. Что Вы хотите предпринять? text_issue_category_reassign_to: Переназначить задачи для данной категории text_issue_updated: Задача %s была обновлена (%s). @@ -755,14 +756,12 @@ text_journal_changed: параметр изменился с %s на %s text_journal_deleted: удалено text_journal_set_to: параметр изменился на %s text_length_between: Длина между %d и %d символов. -text_load_default_configuration: Загрузить конфигурацию по-умолчанию text_load_default_configuration: Загрузить конфигурацию по умолчанию text_min_max_length_info: 0 означает отсутствие запретов text_no_configuration_data: "Роли, трекеры, статусы задач и оперативный план не были сконфигурированы.\nНастоятельно рекомендуется загрузить конфигурацию по-умолчанию. Вы сможете её изменить потом." text_not_assigned: Не назначено text_plugin_assets_writable: Plugin assets directory writable text_project_destroy_confirmation: Вы настаиваете на удалении данного проекта и всей относящейся к нему информации? -text_project_identifier_info: 'Строчные буквы (a-z), допустимы цифры и дефис.<br />Сохраненный идентификатор не может быть изменен.' text_project_identifier_info: 'Допустимы строчные буквы (a-z), цифры и дефис.<br />Сохраненный идентификатор не может быть изменен.' text_reassign_time_entries: 'Перенести зарегистрированное время на следующую задачу:' text_regexp_info: напр. ^[A-Z0-9]+$ @@ -781,11 +780,12 @@ text_tip_task_begin_end_day: начало задачи и окончание е text_tip_task_end_day: дата завершения задачи text_tracker_no_workflow: Для этого трекера последовательность действий не определена text_unallowed_characters: Запрещенные символы -text_user_mail_option: "Для невыбранных проектов, вы будете получать уведомления только о том что просматриваете или в чем участвуете (например, вопросы автором которых вы являетесь или которые вам назначены)." text_user_mail_option: "Для невыбранных проектов, Вы будете получать уведомления только о том что просматриваете или в чем участвуете (например, вопросы, автором которых Вы являетесь или которые Вам назначены)." text_user_wrote: '%s написал(а):' -text_wiki_destroy_confirmation: Вы уверены, что хотите удалить данную вики и все содержимое? text_wiki_destroy_confirmation: Вы уверены, что хотите удалить данную Wiki и все ее содержимое? +text_wiki_page_added: Страница Wiki была добавлена пользователем %s +text_wiki_page_changed: Страница Wiki была %s пользователем %s +text_with_comment: с комментарием '%s' text_workflow_edit: Выберите роль и трекер для редактирования последовательности состояний warning_attachments_not_saved: "%d file(s) could not be saved." lang/ru.yml @@ -203,7 +203,28 @@ class MailerTest < Test::Unit::TestCase assert Mailer.deliver_message_posted(message, recipients) end end + + wiki_remove_tester = lambda do |page, context| + context.assert Mailer.deliver_wiki_remove(page, User.find(1)) + end + [:add, :edit, [:remove, wiki_remove_tester]].each do |action, custom_tester| + define_method 'test_wiki_%s' % action do + page = WikiPage.find(1) + GLoc.valid_languages.each do |lang| + Setting.default_language = lang.to_s + [true, false].each do |plain| + Setting.plain_text_mail = plain + if custom_tester + custom_tester.call(page, self) + else + assert Mailer.send('deliver_wiki_%s' % action, page.content) + end + end + end + end + end + def test_account_information user = User.find(:first) GLoc.valid_languages.each do |lang| test/unit/mailer_test.rb c8aab89b1d1de59dc6648ebc0c69f13eb102f8e2 Artem Vasiliev artem.vasiliev@texunatech.com http://github.com/artemv/redmine_tt/commit/504e44c68b6d7580c3e5b415e549e75ab94ef39d 504e44c68b6d7580c3e5b415e549e75ab94ef39d 2009-01-21T07:20:54-08:00 2009-01-21T07:20:54-08:00 Wiki notification on: * creation * removal * edit 5edc6e2720b0937b1e4aa0350f40ef47dc10a3ad Artem Vasiliev artem.vasiliev@texunatech.com