public-offering
/
pad.pub0.org
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't try validating JSON-ish settings in /admin/settings 

fixes #1965
fix/admin-settings-json
Marcel Klehr 2013-11-19 13:46:16 +01:00
parent 106661712d
commit fc5ec1922a
1 changed files with 7 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
src/static/js/admin/settings.js
View File

@ -13,28 +13,18 @@ $(document).ready(function () {
socket.on('settings', function (settings) {
/* Check to make sure the JSON is clean before proceeding */
if(isJSONClean(settings.results))
{
$('.settings').append(settings.results);
$('.settings').focus();
$('.settings').autosize();
}
else{
alert("YOUR JSON IS BAD AND YOU SHOULD FEEL BAD");
}
// (removed JSON validation, cause it's not JSON!)
$('.settings').append(settings.results);
$('.settings').focus();
$('.settings').autosize();
});
/* When the admin clicks save Settings check the JSON then send the JSON back to the server */
$('#saveSettings').on('click', function(){
var editedSettings = $('.settings').val();
if(isJSONClean(editedSettings)){
// JSON is clean so emit it to the server
socket.emit("saveSettings", $('.settings').val());
}else{
alert("YOUR JSON IS BAD AND YOU SHOULD FEEL BAD")
$('.settings').focus();
}
// (removed JSON validation, cause it's not JSON!)
// emit it to the server
socket.emit("saveSettings", $('.settings').val());
});
/* Tell Etherpad Server to restart */
@ -51,20 +41,3 @@ $(document).ready(function () {
socket.emit("load"); // Load the JSON from the server
});
function isJSONClean(data){
var cleanSettings = JSON.minify(data);
try{
var response = jQuery.parseJSON(cleanSettings);
}
catch(e){
return false; // the JSON failed to be parsed
}
if(typeof response !== 'object'){
return false;
}else{
return true;
}
}