[#MBS-4824] URLCleanup.js only strips whitespace from the front of strings - MusicBrainz JIRA

MusicBrainz Server

Created: 02/Jun/12 08:19 PM Updated: 08/Jun/12 03:05 PM Resolved: 08/Jun/12 03:05 PM

Component/s:

None

Affects Version/s:

None

Fix Version/s:

Bug fixes, 2012-06-11

Description

« Hide

And I quote:

self.cleanUrl = function (sourceType, dirtyURL) {
dirtyURL = dirtyURL.replace(/^\s+/, '');

for (var group in MB.constants.CLEANUPS) {
if(!MB.constants.CLEANUPS.hasOwnProperty(group)) { continue; }

var cleanup = MB.constants.CLEANUPS[group];
if(!cleanup.hasOwnProperty('clean') || !cleanup.match.test(dirtyURL))
continue;

return cleanup.clean(dirtyURL);
}
return dirtyURL;
};

Presumably needs a line:

dirtyURL = dirtyURL.replace(/\s+$/, '');

or some way of combining that into the existing line

Sort Order:

Nicolás Tamargo made changes - 02/Jun/12 08:21 PM

Field	Original Value	New Value
Assignee		Nicolás Tamargo [ reosarevok ]

Nicolás Tamargo made changes - 02/Jun/12 09:19 PM

Status

Open [ 1 ]

Review Submitted [ 5 ]

Oliver Charles made changes - 08/Jun/12 03:05 PM