FIX: Provide the ability to reduce cooked content

This allows us to strip polls from the group posts page.
This commit is contained in:
Robin Ward 2016-04-13 13:24:39 -04:00
parent 8fcd359e2a
commit e91379916b
No known key found for this signature in database
GPG key ID: 0E091E2B4ED1B83D
5 changed files with 25 additions and 19 deletions

View file

@ -51,7 +51,7 @@ module UserNotificationsHelper
def email_excerpt(html, posts_count)
# only include 1st paragraph when more than 1 posts
html = first_paragraph_from(html).to_s if posts_count > 1
raw format_for_email(html)
PrettyText.format_for_email(html).html_safe
end
def normalize_name(name)
@ -65,8 +65,9 @@ module UserNotificationsHelper
normalize_name(post.user.name) != normalize_name(post.user.username)
end
def format_for_email(html)
PrettyText.format_for_email(html).html_safe
def format_for_email(post, use_excerpt)
html = use_excerpt ? post.excerpt : post.cooked
PrettyText.format_for_email(html, post).html_safe
end
end

View file

@ -31,6 +31,12 @@ class GroupPostSerializer < ApplicationSerializer
object.topic
end
def cooked
fragment = Nokogiri::HTML.fragment(object.cooked)
DiscourseEvent.trigger(:reduce_cooked, fragment, object)
fragment.to_html
end
def category
object.topic.category
end

View file

@ -23,7 +23,7 @@
</td>
</tr>
<tr>
<td class='body'><%= format_for_email(use_excerpt ? post.excerpt : post.cooked) %></td>
<td class='body'><%= format_for_email(post, use_excerpt) %></td>
</tr>
</tbody>
</table>

View file

@ -390,8 +390,9 @@ module PrettyText
doc.css(".lightbox-wrapper .meta").remove
end
def self.format_for_email(html)
def self.format_for_email(html, post = nil)
doc = Nokogiri::HTML.fragment(html)
DiscourseEvent.trigger(:reduce_cooked, doc, post)
make_all_links_absolute(doc)
strip_image_wrapping(doc)
doc.to_html

View file

@ -22,19 +22,6 @@ DEFAULT_POLL_NAME ||= "poll".freeze
after_initialize do
# turn polls into a link in emails
Email::Styles.register_plugin_style do |fragment, opts|
post = Post.find_by(id: opts[:post_id]) rescue nil
if post.nil? || post.trashed?
fragment.css(".poll").each(&:remove)
else
post_url = "#{Discourse.base_url}#{post.url}"
fragment.css(".poll").each do |poll|
poll.replace "<p><a href='#{post_url}'>#{I18n.t("poll.email.link_to_poll")}</a></p>"
end
end
end
module ::DiscoursePoll
class Engine < ::Rails::Engine
engine_name PLUGIN_NAME
@ -281,7 +268,7 @@ after_initialize do
self.errors.add(:base, I18n.t("poll.default_poll_with_multiple_choices_has_invalid_parameters")) :
self.errors.add(:base, I18n.t("poll.named_poll_with_multiple_choices_has_invalid_parameters", name: poll["name"]))
return
end
end
end
# store the valid poll
@ -367,6 +354,17 @@ after_initialize do
user ? [POLLS_CUSTOM_FIELD, VOTES_CUSTOM_FIELD] : [POLLS_CUSTOM_FIELD]
end
on(:reduce_cooked) do |fragment, post|
if post.nil? || post.trashed?
fragment.css(".poll, [data-poll-name]").each(&:remove)
else
post_url = "#{Discourse.base_url}#{post.url}"
fragment.css(".poll, [data-poll-name]").each do |poll|
poll.replace "<p><a href='#{post_url}'>#{I18n.t("poll.email.link_to_poll")}</a></p>"
end
end
end
# tells the front-end we have a poll for that post
on(:post_created) do |post|
next if post.is_first_post? || post.custom_fields[POLLS_CUSTOM_FIELD].blank?