Last modified: 2014-02-05 23:58:07 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T62785, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 60785 - Flow: archive search in page header triggers error
Flow: archive search in page header triggers error
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Flow (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 60816 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-03 21:46 UTC by Quiddity
Modified: 2014-02-05 23:58 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
screenshot of error (182.67 KB, image/png)
2014-02-03 21:46 UTC, Quiddity
Details
screenshot 2 (219.70 KB, image/png)
2014-02-04 06:55 UTC, Quiddity
Details

Description Quiddity 2014-02-03 21:46:41 UTC
Created attachment 14462 [details]
screenshot of error

If I have {{archives}} or {{Talk header}} in the page header area, Flow gives this error (screenshot):

The error message received was: Exception Caught: ID searchbox already defined
Comment 1 Quiddity 2014-02-04 06:55:22 UTC
Created attachment 14474 [details]
screenshot 2

After reading the error in the preview (screenshot 1), I removed the {{archives}} box, and hit save. That resulted in this: (screenshot 2)

So I re-edited the header to remove {{Talk header}}. Once I hit save for that, it was still displaying the same error, until I did shift-F5 (hard-refresh).
Comment 2 spage 2014-02-04 07:36:56 UTC
Parsoid is being strict about disallowing the duplicate element IDs in HTML (bug 60816).

But these templates should not be causing invalid HTML output by specifying id=searchbox twice.  Somewhere besides {{archives |search=yes}} there must be another id=searchbox; I tried adding the template wikitext to [[wp:Wikipedia_talk:Flow/Developer_test_page]] but couldn't reproduce it. Assuming that's the problem these templates should use a unique ID, maybe we can alert their authors. Any special styling or behavior they're getting should be due to classes, not an ID.

For grins, you could try to add the problematic templates in VisualEditor.
Comment 3 Erik Bernhardson 2014-02-04 18:11:47 UTC
*** Bug 60816 has been marked as a duplicate of this bug. ***
Comment 4 Gerrit Notification Bot 2014-02-04 18:11:55 UTC
Change 111224 had a related patch set uploaded by EBernhardson:
Don't error out on duplicate id's

https://gerrit.wikimedia.org/r/111224
Comment 5 Quiddity 2014-02-05 03:13:53 UTC
I think the "id=searchbox" is coming from https://www.mediawiki.org/wiki/Extension:InputBox and there are frequently more than one searchbox per page so that does need to be solved.

aha! bug 11777 (Multiple inputboxes make page invalid due to repetitive similar id) I don't know if that bug needs to be poked, with this fresh example? It's been silent since Sept 2012.
Comment 6 Gerrit Notification Bot 2014-02-05 22:46:29 UTC
Change 111224 merged by jenkins-bot:
Don't error out on duplicate id's

https://gerrit.wikimedia.org/r/111224

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


Navigation
Links