[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 Thomas Bleher thomasble...@gmx.de changed: What|Removed |Added Status|NEW |RESOLVED Resolution||FIXED --- Comment #20 from Thomas Bleher thomasble...@gmx.de 2011-02-06 10:04:52 UTC --- Thanks, that fixes the issue for me! -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #19 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-02-06 01:39:24 UTC --- I tried making a change in r81583 to hide these markers from tidy. See if this fixed bug. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #16 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-02-03 23:44:26 UTC --- Are you still seeing this bug in trunk? I could swear I still have trunk installed, I don't have any special local changes to fix the bug left, but I can't reproduce the bug anymore. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #17 from Thomas Bleher thomasble...@gmx.de 2011-02-04 07:18:16 UTC --- Still happens with r81507. $wgUseTidy = false -- everything is OK $wgUseTidy = true -- headings messed up as described -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #18 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-02-04 07:49:29 UTC --- Hmmm, ok, either this has something to do with me upgrading some stuff on my server probably not or there's something wrong with the output in the pages I'm using to test on my test wiki causing XHTML errors tidy can't handle and instead results in tidy being ignored... guess I need to play with test pages. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #13 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-01-29 11:08:19 UTC --- Experimenting a little, Tidy seams to completely ignore the fact that xml and xml namespaces are valid in xhtml and decide to strip out valid namespaced xml anyways. So even when using mw:editsection and defining a valid xhtml namespace in the code we send to tidy, it still decides to ignore that and strip it out. The only thing I've seen to have an effect is adding 'input-xml: yes' to the tidy.conf, however that might have other unwanted effects someone who knows tidy better should point out. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #14 from Thomas Bleher thomasble...@gmx.de 2011-01-29 12:51:53 UTC --- Hmm, so there seem to be several ways forward: 1) Modify tidy config and add 'input-xml: yes' 2) Remove $wgUseTidy, and use the non-tidy-mode in MediaWiki 3) Adapt the parser and the LanguageConverter so they can work together, without interfering with tidy 4) Revert the feature for now Option 1 sounds a bit dangerous, as tidy is explicitly used to clean up faulty markup; using an XML parser that doesn't correct errors seems wrong in this context (The option is described as This option specifies if Tidy should use the XML parser rather than the error correcting HTML parser.) That said: I haven't experimented with this config option, so I may be completely wrong on this point. Option 2 seems quite unrealistic (see eg bug 12221 for what can happen without tidy installed). This change could break page rendering in lots of wrong-but-worked-until-now cases. I do not know the code well enough to say if option 3 is viable, but from the comments so far it doesn't seem so easy. IMHO, it would therefore be best to go for option 4 and revert the changes for now, until a better solution is found. This would at least make trunk work again in this common configuration. Any other options/opinions? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #15 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-01-29 13:14:13 UTC --- 5) Make tidy run on the parser output text each time it's fetched, instead of running on the text before saving it to the parser output Though looking at the code we have ParserBeforeTidy and ParserAfterTidy and trying to remove tidy from Parser::parse might not be totally desirable. 6) Hack up MWTidy so that it replaces things like mw:editsection with something like the parser's UNIQ tokens, runs tidy (making tidy ignore the tokens) and then replaces the tokens with the original blocks. May have some effects like making tidy not work inside anything inside a mw:... block, and it might fall apart with nesting, but it's probably the best option. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 Mark A. Hershberger m...@everybody.org changed: What|Removed |Added CC||m...@everybody.org --- Comment #11 from Mark A. Hershberger m...@everybody.org 2011-01-22 03:43:40 UTC --- Daniel, how did tidy cause this? Can you track it down? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #12 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-01-22 05:24:09 UTC --- how tidy is causing the issue is fairly obvious without even browsing the code. It sounds like tidy is being run after we insert the editsection tokens, but before we replace them with real editsection tokens. The editsection tokens are made in the format editsection page=... section=../editsection specifically because that's the only way to get the LanguageConverter to treat the information in the exact way we want it to. However, unfortunately editsection is not a real html tag, so when tidy sees it before we replace it with editsection markup (which we want to let be customized) it decides to strip out the tag part and leave nothing but the heading hint behind, which is usually the same text as the header's text, leading to that text being displayed twice instead of having a header and an editsection link. Now the solution is less obvious. Perhaps I'll experiment and see if I can use something like a mw: or mediawiki: xml/xhtml namespace and use something like mw:editsection [...] instead of editsection which is in the default xhtml/html namespace and trick tidy into leaving those tags alone. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #9 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-01-13 08:13:29 UTC --- (In reply to comment #8) Created attachment 7985 [details] LocalSettings.php file which exhibits the problem In the hope of making progress on this bug: here is a (slightly redacted for passwords and such) version of LocalSettings.php, with which I can reliably reproduce the bug. I tested without any extensions, and the bug is still there. Any ideas what could cause the bug? I tried some of those settings related to the parser but didn't have any luck. Try disabling tidy to see if that's causing the issue. I tried using the same setting but I'm not sure if I actually have tidy configured on my server. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #10 from Thomas Bleher thomasble...@gmx.de 2011-01-13 14:56:14 UTC --- (In reply to comment #9) Try disabling tidy to see if that's causing the issue. I tried using the same setting but I'm not sure if I actually have tidy configured on my server. Yes, this is it! When disabling tidy, everything works correctly. So the next step is probably to find out why tidy messes up the page this way... -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #8 from Thomas Bleher thomasble...@gmx.de 2011-01-13 07:53:40 UTC --- Created attachment 7985 -- https://bugzilla.wikimedia.org/attachment.cgi?id=7985 LocalSettings.php file which exhibits the problem In the hope of making progress on this bug: here is a (slightly redacted for passwords and such) version of LocalSettings.php, with which I can reliably reproduce the bug. I tested without any extensions, and the bug is still there. Any ideas what could cause the bug? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #1 from Thomas Bleher thomasble...@gmx.de 2011-01-09 11:14:03 UTC --- Created attachment 7963 -- https://bugzilla.wikimedia.org/attachment.cgi?id=7963 Screenshot showing the correct behaviour -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #2 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-01-09 11:25:51 UTC --- Link to url with the issue please. I can't identify the cause of the issue without page source. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 Niklas Laxström niklas.laxst...@gmail.com changed: What|Removed |Added CC||niklas.laxst...@gmail.com --- Comment #3 from Niklas Laxström niklas.laxst...@gmail.com 2011-01-09 11:32:05 UTC --- And... does the problem occur without the script? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #4 from Thomas Bleher thomasble...@gmx.de 2011-01-09 11:48:13 UTC --- The page that is shown in the screenshot is http://spiele.j-crew.de/wiki/Technikworkshop . I have however already reverted to the old version because of the bug (so you won't see the issue directly, but you can examine the page source). It occured on multiple pages, so it doesn't seem too page specific. I should also note that I had to explicitly purge the page to see the new behaviour, which I found a bit strange - after reading the commit messages, I had assumed that old parsed pages would be evicted from the cache automatically. I downloaded the page using wget, and the headings are already doubled in the page source so it seems to be unrelated to the Javascript code I posted. Old version of first header: h2span class=editsection[a href=/w/index.php?title=Technikworkshopamp;action=editamp;section=1 title=Abschnitt bearbeiten: KurzbeschreibungBearbeiten/a]/span span class=mw-headline id=Kurzbeschreibunga href=/wiki/Datei:Info.png class=imageimg alt= src=/images/b/b3/Info.png width=32 height=32 //a Kurzbeschreibung/span/h2 New version of the first header: h2Kurzbeschreibung span class=mw-headline id=Kurzbeschreibunga href=/wiki/Datei:Info.png class=imageimg alt= src=/images/b/b3/Info.png width=32 height=32 //a Kurzbeschreibung/span/h2 As you can see, the word Kurzbeschreibung is doubled. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #5 from Thomas Bleher thomasble...@gmx.de 2011-01-09 11:49:52 UTC --- Created attachment 7965 -- https://bugzilla.wikimedia.org/attachment.cgi?id=7965 The correct html output, from phase3 r79519 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #6 from Thomas Bleher thomasble...@gmx.de 2011-01-09 11:51:03 UTC --- Created attachment 7966 -- https://bugzilla.wikimedia.org/attachment.cgi?id=7966 The incorrect html output, from phase3 r79878 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 26639] Headings appear twice, instead of edit section links
https://bugzilla.wikimedia.org/show_bug.cgi?id=26639 --- Comment #7 from Daniel Friesen mediawiki-b...@nadir-seen-fire.com 2011-01-09 11:56:52 UTC --- Hmmm... this is strange behavior... It seams like for some reason something is stripping the editsection [...]Text/editsection down to Text. But everything worked fine when I coded it... parser tests to. and those included stuff in other languages. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
