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) {
if (value.trim().length === 0) {
return "hide";
} else {
return "show";
}
return value.trim() ? "show" : "hide";
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-01-15 06:37:17 +00:00
Outdated
Review
Copy Link

Si los valores van a ser 2 y tan cortos, quizas lo cambiaría a ternario

Si los valores van a ser 2 y tan cortos, quizas lo cambiaría a ternario
}
function ifIsEmptyImage(value) {
if (value.trim().length === 0) {
return "img/in.svg";
} else {
return "img/" + value + ".svg";
}
return value.trim() ? `img/${value}.svg` :"img/in.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) {
return value;
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.toString().trim() ? value : "00:00";
}
function cerrar(){