Changeset 32773

Timestamp:

2019-02-13T17:48:56+13:00 (5 years ago)

Author:

ak19

Message:

Minor change: optional parameter encode_percentages also introduced for makeSafeForURL, where it defaults to true this time

File:

• main/trunk/greenstone3/web/interfaces/default/js/document_scripts.js (modified) (2 diffs)

main/trunk/greenstone3/web/interfaces/default/js/document_scripts.js

@@ -42 +42 @@
 */
 /* URL encode RESERVED characters in a non-URL context of a URL, such as the inline template (ilt) parameter value of a URL */
-function makeSafeForURL(url_part) {
+function makeSafeForURL(url_part, encode_percentages) {
     // https://stackoverflow.com/questions/7368407/javascript-replace-a-set-of-characters-with-another-one
     var reserved_mappings = {
@@ -53 +53 @@
     '&': '%26'
     };
-    
-    encode_percentages = 1; // to force the URL-encoding of any % in url_part, do this for inline-templates that haven't ever been encoded
+    // https://stackoverflow.com/questions/12797118/how-can-i-declare-optional-function-parameters-in-javascript
+    encode_percentages = encode_percentages || 1; // this method forces the URL-encoding of any % in url_part, e.g. do this for inline-templates that haven't ever been encoded
     url_part = makeURLSafe(url_part, encode_percentages);