View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0016562||phpList 3 application||Bounce Management||public||24-03-12 03:26||07-06-13 21:30|
|Status||resolved||Resolution||no change required|
|Summary||0016562: File bounce php|
|Description||and command : DeletingB line 11 in the file.|
Message read is : Deleting bounce 56 ..\n..Done, ...
after 56, there is something which does not fit : ..\n..
It should stop at 56 in the exemple, put a dot and then go on with Done etc..
It should be : Deleting boune 56. Done etc...
|Tags||No tags attached.|
||The \n is intended to put the message on a line on its own. It needs to be in double quotes, not single, "\n".|
||Lines 75 and 79 of file admin/bounce.php need to be changed.|
In Bounce.php in lang directory it is written as follows : line 11
"DeletingB" => "Deleting bounce %s ..\n",
If you put the n between double quotes, it won't fit, because the " is already used for delimitation instead of the '.
Or is it to write line 75 and 79 as follows ?
75 as print sprintf($GLOBALS['I18N']->get('DelUser')."\n", $userid);
75 written now :print sprintf($GLOBALS['I18N']->get('DelUser').'\n', $userid);
79print sprintf($GLOBALS['I18N']->get('DeletingB')."\n", $id);
79 written now :print sprintf($GLOBALS['I18N']->get('DeletingB').'\n', $id);
Ah, as usual it is more complex than I thought.
As you reported, there is a literal \n in the message, so lines 75 and 79 need to be changed to either remove it or change it to "\n" for a new-line.
But the DeletingB message text already includes a \n correctly inside double-quotes, so it is probably not needed at all on line 79.
The developers need to decide on the best solution.
Well, in my opinion it is not compulsory to change line for the message.
I have fixed it as follows :
"DeletingB" => "Suppression de l'envoi numéro %s revenu en erreur.",
I translate :
"DeletingB" => "Deleting sending-message having the number %s which came back in error.",
Well for once it is easier to write in french than to translate into english.
In fact you do a message, then the same message has a number according to the number of people to who it is sent. If you send it to two people, then you have message-sent n° 1 then message-sent n° 2.
Well after changing in the bounce fr file, I have made an other trial.
It comes out as follows :
Suppression de l'envoi numéro 57 revenu en erreur.\n. .C'est fait, chargement du prochain message revenu en erreur.
retour vers la liste d'emails rejetés
In fact, on that point too, the translation is not correct, it should be messages returned.
And it comes out that the changes must be done in the file bounce under admin, because I have deleted the n wich was included in the translation.
This problem is resolved by someone who has given me the solution as follows :
line 79 of bounce.php is now written as follows :
print sprintf($GLOBALS['I18N']->get('DeletingB'), $id);echo "\n";
and it works perfectly, so you can include it and make the appropriate correction