Bugzilla@Mozdev – Bug 26512
Second use of %matchTextFromSubject()% is blank
Last modified: 2018-04-12 03:59:29
You need to log in before you can comment on or make changes to this bug.
Created an attachment (id=8434) [details] Result of the template (blank result in the circle) I have a template where i use ... to grab the last word in the original subject (this is formatted "Smartlist aanvraag voor aaa-bbb" My Template: L.S.,<br> <br> De door u gevraagde maillijst %matchTextFromSubject("(?:\S+\s+){3}(\S+)",1)%@amc.nl is aangemaakt en is beschikbaar voor gebruik.<br> <br> Voor het beheer van uw maillijst is er een intuitieve en gebruikersvriendelijke beheerstool gemaakt die beschikbaar is via<br> <br> http://listserver.amc.nl/interface/list.php?list=%matchTextFromSubject("(?:\S+\s+){3}(\S+)",1)%<br> <br> Om deze tool te kunnen gebruiken hebt u wel een webbrowser nodig zoals Mozilla Firefox, Google Chrome of Microsoft Internet Explorer.<br> <br> Lijstje van handige url's (url=adres van een webpagina) waar u eventueel een bookmark van kan maken in uw webbrowser:<br> <br> Een overzicht van al de door u beheerde lijsten:<br> http://listserver.amc.nl/interface/mylists.php?beheerder=%from(mail)%<br> <br> Algemene home page van de listserver:<br> http://listserver.amc.nl/<br> <br> Toon alle beschikbare lijsten:<br> http://listserver.amc.nl/beheer/show<br> <br> BELANGRIJKE NOOT:<br> De gebruikte software voor de listserver gaat niet goed om met mailtjes die in HTML formaat zijn opgemaakt. Maakt u de abonnees van uw lijst hierop attent. Attachments zijn toegestaan, maar de "body" van de mail moet in platte text verstuurd worden.<br> <br> %sig(text)% In the template is two identical uses of %matchTextFromSubject()%, the first one results in aaa-bbb, the second one stays blank. I cut and pasted the first variable to make sure they are identical, but no change, the second use stays blank. Attached a screenshot (slightly modified to protect privacy) with a circle around the empty second use of the variable.
(In reply to comment #0) > Created an attachment (id=8434) [details] [details] > Result of the template (blank result in the circle) > > I have a template where i use ... to grab the last word in the original subject > (this is formatted "Smartlist aanvraag voor aaa-bbb" > > My Template: > L.S.,<br> > <br> > De door u gevraagde maillijst > %matchTextFromSubject("(?:\S+\s+){3}(\S+)",1)%@amc.nl is aangemaakt en is > beschikbaar voor gebruik.<br> > <br> > Voor het beheer van uw maillijst is er een intuitieve en gebruikersvriendelijke > beheerstool gemaakt die beschikbaar is via<br> > <br> > > http://listserver.amc.nl/interface/list.php?list=%matchTextFromSubject("(?:\S+\s+){3}(\S+)",1)%<br> > <br> > Om deze tool te kunnen gebruiken hebt u wel een webbrowser nodig zoals > Mozilla Firefox, Google Chrome of Microsoft Internet Explorer.<br> > <br> > Lijstje van handige url's (url=adres van een webpagina) waar u > eventueel een bookmark van kan maken in uw webbrowser:<br> > <br> > Een overzicht van al de door u beheerde lijsten:<br> > http://listserver.amc.nl/interface/mylists.php?beheerder=%from(mail)%<br> > <br> > Algemene home page van de listserver:<br> > http://listserver.amc.nl/<br> > <br> > Toon alle beschikbare lijsten:<br> > http://listserver.amc.nl/beheer/show<br> > <br> > BELANGRIJKE NOOT:<br> > De gebruikte software voor de listserver gaat niet goed om met mailtjes > die in HTML formaat zijn opgemaakt. Maakt u de abonnees van uw lijst > hierop attent. Attachments zijn toegestaan, maar de "body" van de mail moet in > platte text verstuurd worden.<br> > <br> > %sig(text)% > I tested by replying to an email with subject line "Smartlist creation request for aaa-bbb" and both instances returned nothing. I think the regex is a little complicated though, why do you not change it to something that doesn't "count words" "Smartlist creation request for (\S+)" you can use it like this %matchTextFromSubject("Smartlist creation request for (\S+)",1)% What appears to happen is that only works every "other" time - 1st time 3rd time etc. so I think there is something wrong with the regex matching when finding the instance of the matchtextfromsubject variable in the original text. Again, I need to debug this to find out.
I found the cause of the problem. I mistyped a piece of code (indented without curly brackets). this leads to every 2nd instance of the matched regex being replaced with an empty string: else removePat = true; msg = msg.replace(matchPart[i],""); CORRECTED CODE: else { removePat = true; msg = msg.replace(matchPart[i],""); }
Created an attachment (id=8435) [details] 1.5 prerelease 199 Bugfixed version. Good catch! My test stationery, where I also was trying to add the functional link, which worked: <p> L.S., </p> <p> De door u gevraagde maillijst <b>%matchTextFromSubject("Smartlist creation request for (\S+)",1)%@amc.nl</b> is aangemaakt en is beschikbaar voor gebruik. </p> <p> Voor het beheer van uw maillijst is er een intuitieve en gebruikersvriendelijke beheerstool gemaakt die beschikbaar is via </p> <p> <a href="http://listserver.amc.nl/interface/list.php?list=%matchTextFromSubject("Smartlist creation request for (\S+)",1)%"> http://listserver.amc.nl/interface/list.php?list=%matchTextFromSubject("Smartlist creation request for (\S+)",1)%</a> </p> <p> Om deze tool te kunnen gebruiken hebt u wel een webbrowser nodig zoals Mozilla Firefox, Google Chrome of Microsoft Internet Explorer. </p> <p> Lijstje van handige url's (url=adres van een webpagina) waar u eventueel een bookmark van kan maken in uw webbrowser: </p> <p> Een overzicht van al de door u beheerde lijsten:<br> http://listserver.amc.nl/interface/mylists.php?beheerder=%from(mail)% </p> <p> Algemene home page van de listserver:<br> http://listserver.amc.nl/ </p> <p> Toon alle beschikbare lijsten:<br> http://listserver.amc.nl/beheer/show </p> <p> BELANGRIJKE NOOT:<br> De gebruikte software voor de listserver gaat niet goed om met mailtjes die in HTML formaat zijn opgemaakt. Maakt u de abonnees van uw lijst hierop attent. Attachments zijn toegestaan, maar de "body" van de mail moet in platte text verstuurd worden. </p> %sig(text)%
Resolving to FIXED. I will update [Bug 26411] with the new version, too.
Thanks for the fix, can confirm it now happily works for me :)