subject:"\"\\\[Bug 57458\\\] '\\\\n' are added to various elements in CommonsMetadata output\""

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-09-05 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #16 from Tisza Gergő --- *** Bug 66652 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-23 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 Tisza Gergő changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED Resolution|---

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-23 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #14 from Gerrit Notification Bot --- Change 155901 merged by jenkins-bot: TemplateParser: Fix whitespace trim https://gerrit.wikimedia.org/r/155901 -- You are receiving this mail because: You are on the CC list for the bug. _

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-23 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #13 from Gerrit Notification Bot --- Change 155901 had a related patch set uploaded by TheDJ: TemplateParser: Fix whitespace trim https://gerrit.wikimedia.org/r/155901 -- You are receiving this mail because: You are on the CC lis

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-23 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 Gerrit Notification Bot changed: What|Removed |Added Status|REOPENED|PATCH_TO_REVIEW -- You are

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-14 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #12 from Lupo --- (In reply to Tisza Gergő from comment #10) > Also, Lokal_Profil is right that the newline is > not always present in the HTML code. I'll test locally with the examples > mentioned here. Not correct. See https://c

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-14 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #11 from Lupo --- This code does _not_ look good. '/^\s+(.*)\s+$/' is wrong. It fails to trim if there are no leading blanks (or no trailing blanks). And watch out for the greedy (.*), that also looks wrong. -- You are receiving t

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-14 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #10 from Tisza Gergő --- The code to remove is in https://gerrit.wikimedia.org/r/#/c/120948/1/TemplateParser.php which at a glance seems correct to me. Also, Lokal_Profil is right that the newline is not always present in the HTML c

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-14 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #9 from Lupo --- As stated in bug 69497, these newlines are in the license template, and the code doing the HTML scraping there had better remove them. -- You are receiving this mail because: You are on the CC list for the bug. __

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-14 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 Tisza Gergő changed: What|Removed |Added CC||lupo.bugzi...@gmail.com --- Comment #8 f

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-08-13 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #7 from Lokal_Profil --- Looking at the html source of the example above [1] there is no trace of these newline characters. Hence it might not be a cleaning/trimming issue in the TemplateParser but rather inserted by it? [1] https:

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-05-21 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 Lokal_Profil changed: What|Removed |Added See Also||https://bugzilla.wikimedia.

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-05-21 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 Lokal_Profil changed: What|Removed |Added Status|RESOLVED|REOPENED CC|

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-03-27 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 Tisza Gergő changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED Resolution|---

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-03-27 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #5 from Gerrit Notification Bot --- Change 120948 merged by jenkins-bot: Clean parsed HTML https://gerrit.wikimedia.org/r/120948 -- You are receiving this mail because: You are on the CC list for the bug.

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2014-03-25 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #4 from Gerrit Notification Bot --- Change 120948 had a related patch set uploaded by Gergő Tisza: Clean parsed HTML https://gerrit.wikimedia.org/r/120948 -- You are receiving this mail because: You are on the CC list for the bug

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2013-12-10 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #3 from Gerrit Notification Bot --- Change 97743 abandoned by Gergő Tisza: Trim HTML-based metadata values Reason: Abandoning this change since InformationParser has been completely rewritten in the meantime. https://gerrit.wikime

[Bug 57458] '\n' are added to various elements in CommonsMetadata output

2013-11-26 Thread bugzilla-daemon

https://bugzilla.wikimedia.org/show_bug.cgi?id=57458 --- Comment #2 from Gerrit Notification Bot --- Change 97743 had a related patch set uploaded by Gergő Tisza: Trim HTML-based metadata values https://gerrit.wikimedia.org/r/97743 -- You are receiving this mail because: You are on the CC list