Sputniza has uploaded a new change for review.
https://gerrit.wikimedia.org/r/183998
Change subject: Change slashes for dashes
......................................................................
Change slashes for dashes
Instead of giving an error message, this replaces '/' with '-' in the course
name,
to keep it from causing database errors.
Bug: T74889
Change-Id: I8a9ba843275d20627077bd95fb5849a95762fa4b
---
M resources/ep.addcourse.js
M resources/ep.addorg.js
2 files changed, 2 insertions(+), 5 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/EducationProgram
refs/changes/98/183998/1
diff --git a/resources/ep.addcourse.js b/resources/ep.addcourse.js
index 15d8272..6dfa509 100644
--- a/resources/ep.addcourse.js
+++ b/resources/ep.addcourse.js
@@ -12,10 +12,7 @@
$( '.ep-course-add' ).closest( 'form' ).submit( function(event)
{
var courseName = $( '#newname' ).val();
- if ( courseName.indexOf( "/" ) > -1 ) {
- alert("Please don't use any slashes in your
course name!");
- event.preventDefault();
- }
+ courseName = courseName.replace(/\//g, "-");
courseName = courseName.charAt( 0 ).toUpperCase() +
courseName.slice( 1 );
$( this ).attr(
'action',
diff --git a/resources/ep.addorg.js b/resources/ep.addorg.js
index e37279a..b1a9d6a 100644
--- a/resources/ep.addorg.js
+++ b/resources/ep.addorg.js
@@ -18,4 +18,4 @@
} );
-})( window.jQuery );
\ No newline at end of file
+})( window.jQuery );
--
To view, visit https://gerrit.wikimedia.org/r/183998
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8a9ba843275d20627077bd95fb5849a95762fa4b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/EducationProgram
Gerrit-Branch: master
Gerrit-Owner: Sputniza <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
