Skip to content

Commit

Permalink
[BUG-1] Can't view D20 stats for users with a space in their name
Browse files Browse the repository at this point in the history
Closes #1
  • Loading branch information
Yosoy-Ed committed May 4, 2024
1 parent aab8205 commit 44b29e8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
2 changes: 1 addition & 1 deletion module.json
Expand Up @@ -8,7 +8,7 @@
"flags": {}
}
],
"version": "0.1.2",
"version": "0.1.3",
"styles": [
"css/sds.css"
],
Expand Down
10 changes: 6 additions & 4 deletions scripts/sds.js
Expand Up @@ -73,7 +73,7 @@ class ChartWindow extends Application {
// Changing user
$('#selectuser').change(ev => {

let theuser = $("#selectuser").val();
let theuser = $("#selectuser").val();

let prevfrom = $('select[name="datefrom"] option:selected').text();
let prevto = $('select[name="dateto"] option:selected').text();
Expand Down Expand Up @@ -530,16 +530,18 @@ function wus() {
let whichuser = '';

for (var i = 0; i < (usnames.length); i++) {
whichuser = whichuser + '<option value=' + usnames[i] + '>' + usnames[i] + '</option>';
whichuser = `${whichuser} <option value="${usnames[i]}">${usnames[i]}</option>`;
}

let gmids = game.users.contents.filter(user => user.isGM).map(gm => gm.id);
if (!game.settings.get('simple-dice-stats', 'allowviewgmrolls') && !gmids.includes(game.user._id)) {

for (let x of gmids) {
let gmname = game.users.get(x).name;
let valuetodelete = '<option value=' + gmname + '>' + gmname + '</option>';
let valuetodelete = `<option value="${gmname}">${gmname}</option>`;
console.log(valuetodelete)
whichuser = whichuser.replace(valuetodelete, '');
console.log(whichuser)

}
}
Expand All @@ -557,7 +559,7 @@ function d20icon(theusercolor) {
}

//Update the list of dates based on the selected user
function populatedates(user) {
function populatedates(user) {

let alldates = Object.keys(game.users.contents.find(f => f.name === user)['flags']['simple-dice-stats']['d20stats']);

Expand Down

0 comments on commit 44b29e8

Please sign in to comment.