Issue: PM notification emails broken after upgrade

Issue Tools
- View Changes

March 1, 2011 10:24 AM

ctrlbrk

Junior Member

PM notification emails broken after upgrade

vb 3.8.5

Was using vw 3.0.9. Received email saying critical to upgrade to 3.1.1.

Did so, now all my emails are broken, it is sending html inside the email. This did not happen before going to 3.1.1, and it was the onyl thing changed, and it started happening the moment 3.1.1 was installed. This is critical, the forum sends a thousand emails a day so this is embarrassing to me.

Here is a test PM sent to myself before pressing 'submit':

And the email itself:

You can see it is now inserting HTML markup (br).

And it gets worse / more visible after QUOTING, original:

Email itself:

Mike

Issue Details

Issue Number 2250

Issue Type Bug

Project VaultWiki 3.x Series

Category BB-Code Parsing

Status Fixed

Priority 2 - Fatal / Database Errors

Affected Version 3.0.11

Fixed Version 3.0.12

Milestone (none)

Software DependencyAny

Users able to reproduce bug 0

Users unable to reproduce bug 0

Attachments 0

Assigned Users (none)

Tags (none)

March 1, 2011 12:00 PM

ctrlbrk

Junior Member

Looking more closely, only PM's are broken. Still bad.

Reply

March 1, 2011 2:36 PM

pegasus

VaultWiki Team

Fixed in the current ZIP, build number will be increased tonight.

In vault/class/bbcode.php, find:

Code:

			static $internal;

			if ($internal === null)
			{
				$internal = new vB_BbCodeParser($this->registry, fetch_tag_list());
			}

			if ($this->parse_userinfo)
			{
				$internal->set_parse_userinfo($this->parse_userinfo);
			}

			return $internal->parse($text, $forumid, $allowsmilie, $isimgcheck, $parsedtext, $parsedhasimages, $cachable, $htmlstate);

Replace with:

Code:

			return parent::parse($text, $forumid, $allowsmilie, $isimgcheck, $parsedtext, $parsedhasimages, $cachable, $htmlstate);

March 1, 2011 5:29 PM

ctrlbrk

Junior Member

Can you be more specific, I am replace 12 lines with 1? Or just the return line?

Reply
March 1, 2011 5:34 PM

pegasus

VaultWiki Team

All lines in the Find box should be replaced.

Reply

+ Reply

All times are GMT -4. The time now is 11:50 AM.

This site uses cookies to help personalize content, to tailor your experience, and to keep you logged in if you register.
By continuing to use this site, you are consenting to our use of cookies.

Learn more… Accept Remind me later

Welcome to VaultWiki.org, home of the wiki add-on for vBulletin and XenForo!

Issue: PM notification emails broken after upgrade

Issue Tools