Last modified: 2007-11-29 17:18:19 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 T11849, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 9849 - SelectCategory: previewing clears the categories
SelectCategory: previewing clears the categories
Status: RESOLVED WORKSFORME
Product: MediaWiki extensions
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
PC Linux
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-05-08 21:13 UTC by Samuel Gélineau
Modified: 2007-11-29 17:18 UTC (History)
1 user (show)

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


Attachments
preview_bug.patch (788 bytes, patch)
2007-05-08 21:18 UTC, Samuel Gélineau
Details

Description Samuel Gélineau 2007-05-08 21:13:17 UTC
When using the SelectCategory extension, clicking the Preview button loads the
preview page, but on this page the SelectCategory select box contains no
selections. As a result, if the user does not notice this and does not reselect
the correct categories, subsequently clicking the Save button will cause the
categories to be lost.
Comment 1 Samuel Gélineau 2007-05-08 21:18:29 UTC
Created attachment 3601 [details]
preview_bug.patch

If $_POST["SelectCategoryList"] is defined, this means the categories were
already extracted from the page text in a previous PHP pass. Thus, the
categories listed in this variable should be used. Failing to do so cause the
reextraction to fail, since the first extraction striped the categories from
the page text.
Comment 2 Shannon Lowder 2007-11-29 14:46:46 UTC
Although it was commented that the .5 version of selectCategories resolves this issue, the problem remains.  I have also compared the function suggested by the patch versus the same function definition found in .5, they are a match.  I am still receiving this error.  Were there any other changes required to get version .5 to really resolve this issue?
Comment 3 Samuel Gélineau 2007-11-29 17:18:19 UTC
The stock .5 version works fine on my machine...

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


Navigation
Links