Last modified: 2013-04-15 19:43:56 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 T48873, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 46873 - strip trailing spaces from enrolment token
strip trailing spaces from enrolment token
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
EducationProgram (Other open bugs)
unspecified
All All
: Low minor (vote)
: ---
Assigned To: Jeroen De Dauw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-04 01:25 UTC by Sage Ross
Modified: 2013-04-15 19:43 UTC (History)
2 users (show)

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


Attachments

Description Sage Ross 2013-04-04 01:25:59 UTC
If an instructor accidentally includes trailing spaces in an enrolment token, this can cause confusion and also is not handled correctly by the course page.

For users with the relevant user rights, the enrolment token is visible in the Summary table of a course page, and it links straight to enrolment using that token. However, trailing spaces are not visible do not work through the link, so clicking the link results in an invalid token error. And it is not very apparent to users why the token is incorrect, as cutting and pasting or entering it manually will not include the trailing space either.

I suggest that whitespace simply be stripped off of any enrolment token upon course page creation/update.
Comment 1 Jeroen De Dauw 2013-04-04 01:35:05 UTC
https://gerrit.wikimedia.org/r/#/c/57458/
Comment 2 Jeroen De Dauw 2013-04-04 01:35:20 UTC
How is that for response time? ;p
Comment 3 Sage Ross 2013-04-04 01:35:40 UTC
:-D

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


Navigation
Links