verdnatura/worker-time-control
verdnatura
/
worker-time-control
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

#6656 fix timetable #19

Merged
jorgep merged 2 commits from 6656-fixTimeTable into test 2024-01-17 14:01:06 +00:00
Conversation 0 Commits 2 Files Changed 1 +4 -12
1 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
js/clockIn.js
View File

@ -210,23 +210,15 @@ function printErrores(errores) {
} }
function ifIsEmpty(value) { function ifIsEmpty(value) {
if (value.trim().length === 0) { return value.trim() ? "show" : "hide";
return "hide";
} else {
return "show";
}
} }
function ifIsEmptyImage(value) { function ifIsEmptyImage(value) {
if (value.trim().length === 0) { return value.trim() ? `img/${value}.svg` :"img/in.svg";
return "img/in.svg";
} else {
return "img/" + value + ".svg";
}
} }
jorgep marked this conversation as resolved
jsegarra commented 2024-01-15 06:35:36 +00:00
Review
Copy Link

Lo revisamos porque se podría dejar en una línea.
Ya que construyes un string en la 220, puedes aprovechar a darle un valor por defecto a value que sea in

Lo revisamos porque se podría dejar en una línea. Ya que construyes un string en la 220, puedes aprovechar a darle un valor por defecto a value que sea in
function ifIsEmptyText(value = 0) { function ifIsEmptyText(value) {
jorgep marked this conversation as resolved
jsegarra commented 2024-01-15 06:35:51 +00:00
Review
Copy Link

Se podría usar template string

Se podría usar template string
return value; return value.toString().trim() ? value : "00:00";
} }
function cerrar(){ function cerrar(){