First Last Prev Next    No search results available
Details
: Second use of %matchTextFromSubject()% is blank
Bug#: 26512
: smarttemplate4
: General
Status: RESOLVED
Resolution: FIXED
: PC
: Linux
: unspecified
: P2
: normal
: ---

:
:
:
:
:
  Show dependency tree - Show dependency graph
People
Reporter: Vincent Ouwehand <vincent.ouwehand@gmail.com>
Assigned To: Axel Grude <axel.grude@gmail.com>

Attachments
Result of the template (blank result in the circle) (82.15 KB, image/png)
2018-04-12 00:56, Vincent Ouwehand
no flags Details
1.5 prerelease 199 (463.87 KB, application/x-xpinstall)
2018-04-12 03:46, Axel Grude
no flags Details


Note

You need to log in before you can comment on or make changes to this bug.

Related actions


Description:   Opened: 2018-04-12 00:56
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.
------- Comment #1 From Axel Grude 2018-04-12 02:58:54 -------
(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.
------- Comment #2 From Axel Grude 2018-04-12 03:31:09 -------
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],"");
  }
------- Comment #3 From Axel Grude 2018-04-12 03:46:52 -------
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)%
------- Comment #4 From Axel Grude 2018-04-12 03:49:20 -------
Resolving to FIXED. I will update [Bug 26411] with the new version, too.
------- Comment #5 From Vincent Ouwehand 2018-04-12 03:59:29 -------
Thanks for the fix, can confirm it now happily works for me :)

First Last Prev Next    No search results available