Clean JS in admin to use const/let instead of var

This commit is contained in:
Nabeel Shahzad
2018-02-22 11:47:28 -06:00
parent 98eef1eb12
commit fc381da21b

View File

@@ -102,8 +102,9 @@
<script src="{!! public_asset('/assets/admin/js/admin.js') !!}"></script> <script src="{!! public_asset('/assets/admin/js/admin.js') !!}"></script>
<script> <script>
var getStorage = function(key) { const getStorage = function(key) {
var st = window.localStorage.getItem(key); const st = window.localStorage.getItem(key);
console.log('storage: ', key, st); console.log('storage: ', key, st);
if(_.isNil(st)) { if(_.isNil(st)) {
return { return {
@@ -114,17 +115,16 @@ var getStorage = function(key) {
return JSON.parse(st); return JSON.parse(st);
}; };
var saveStorage = function(key, obj) { const saveStorage = function(key, obj) {
console.log('save: ', key, obj); console.log('save: ', key, obj);
window.localStorage.setItem(key, JSON.stringify(obj)); window.localStorage.setItem(key, JSON.stringify(obj));
}; };
var addItem = function(obj, item) { const addItem = function(obj, item) {
if (_.isNil(obj)) {
obj = [];
}
var index = _.indexOf(obj, item); if (_.isNil(obj)) { obj = []; }
const index = _.indexOf(obj, item);
if(index === -1) { if(index === -1) {
obj.push(item); obj.push(item);
} }
@@ -132,11 +132,11 @@ var addItem = function(obj, item) {
return obj; return obj;
}; };
var removeItem = function (obj, item) { const removeItem = function (obj, item) {
if (_.isNil(obj)) {
obj = []; if (_.isNil(obj)) { obj = []; }
}
var index = _.indexOf(obj, item); const index = _.indexOf(obj, item);
if (index !== -1) { if (index !== -1) {
console.log("removing", item); console.log("removing", item);
obj.splice(index, 1); obj.splice(index, 1);
@@ -154,35 +154,34 @@ $(document).ready(function () {
radioClass: 'icheckbox_square-blue' radioClass: 'icheckbox_square-blue'
}); });
var storage = getStorage("phpvms.admin"); let storage = getStorage("phpvms.admin");
// see what menu items should be open // see what menu items should be open
for(var idx = 0; idx < storage.menu.length; idx++) { for(let idx = 0; idx < storage.menu.length; idx++) {
var id = storage.menu[idx]; const id = storage.menu[idx];
var elem = $(".collapse#" + id); const elem = $(".collapse#" + id);
elem.addClass("in").trigger("show.bs.collapse"); elem.addClass("in").trigger("show.bs.collapse");
var caret = $("a." + id + " b"); const caret = $("a." + id + " b");
caret.addClass("pe-7s-angle-down"); caret.addClass("pe-7s-angle-down");
caret.removeClass("pe-7s-angle-up"); caret.removeClass("pe-7s-angle-up");
} }
$(".collapse").on("hide.bs.collapse", function () { $(".collapse").on("hide.bs.collapse", function () {
console.log('hiding'); console.log('hiding');
var id = $(this).attr('id'); const id = $(this).attr('id');
var elem = $("a." + id + " b"); const elem = $("a." + id + " b");
elem.removeClass("pe-7s-angle-down"); elem.removeClass("pe-7s-angle-down");
elem.addClass("pe-7s-angle-up"); elem.addClass("pe-7s-angle-up");
removeItem(storage.menu, id); removeItem(storage.menu, id);
saveStorage("phpvms.admin", storage); saveStorage("phpvms.admin", storage);
}); });
$(".collapse").on("show.bs.collapse", function () { $(".collapse").on("show.bs.collapse", function () {
console.log('showing'); console.log('showing');
var id = $(this).attr('id'); const id = $(this).attr('id');
var caret = $("a." + id + " b"); const caret = $("a." + id + " b");
caret.addClass("pe-7s-angle-down"); caret.addClass("pe-7s-angle-down");
caret.removeClass("pe-7s-angle-up"); caret.removeClass("pe-7s-angle-up");