Last modified: 2013-12-05 04:50:28 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T56996, the corresponding Phabricator task for complete and up-to-date bug report information.

Bug 54996 - Flow: notification emails have <p> tags around their titles


Summary:	Flow: notification emails have <p> tags around their titles

Status:	RESOLVED FIXED

Product:	MediaWiki extensions
Classification:	Unclassified
Component:	Flow (Other open bugs)
Version:	master
Hardware:	All All

Importance:	Unprioritized normal (vote)
Target Milestone:	---
Assigned To:	Andrew Garrett

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2013-10-04 23:07 UTC by spage
Modified:	2013-12-05 04:50 UTC (History)
CC List:	4 users (show)

See Also:
Web browser:	---
Mobile Platform:	---
Assignee Huggle Beta Tester:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description spage 2013-10-04 23:07:07 UTC

I got a notification e-mail from Flow (yay!), the e-mail body appeared as:

Maryana created a new topic with the title "<p>Welcome to testwiki!
</p>" on User talk:Spage

^there's a <br> here before the closing quote in my mail message

The topic title should not be wrapped in a <p> tag, and if the <br>/newline is added by Flow then it should not be present. I think Flow devs worked on a fix for title handling in commit a7c74886bbf.

Comment 1 bsitu 2013-10-05 00:28:41 UTC

The default format for Post::getContent() is html so it's saving the topic title as html format for notification

Comment 2 spage 2013-10-05 01:35:51 UTC

Prioritization and scheduling of this bug is tracked on Mingle card https://mingle.corp.wikimedia.org/projects/flow/cards/296

Wikimedia Bugzilla is closed!

Search

Personal tools

Navigation

Links