Skip to content

Commit

Permalink
Update nextWeekAtWeekend.js
Browse files Browse the repository at this point in the history
  • Loading branch information
@docentYT
docentYT committed Sep 23, 2023
1 parent 7ffe55a commit 5d9f4f2
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions extensions/combined/src/przegladaj_plan_lekcji/nextWeekAtWeekend.js
View file
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
function main() {
let timetable = document.getElementsByClassName("plan-lekcji")[0];
let headers = timetable.getElementsByTagName("thead")[0].getElementsByTagName("tr")[0];
const timetable = document.getElementsByClassName("plan-lekcji")[0];
const headers = timetable.getElementsByTagName("thead")[0].getElementsByTagName("tr")[0];

This comment has been minimized.

Sign in to view

Copy link
@EnterVPL

EnterVPL Sep 27, 2023

NIEBEZPIECZNE

Jeśli kiedykolwiek utworzysz nową tabelkę nad tą, którą tu używasz to będziesz miał błąd w kodzie.

Rozwiązanie

Przypisz IDki do tych elementów.

PS. ClassName też lepiej unikać do pobierania pojedynczego elementu. Do tego używaj zawsze ID bo jest unikalny. Jeżeli będziesz chciał kiedyś operować na grupie elementów to wymyśl własny atrybut i pobieraj po nim elementy.

This comment has been minimized.

Sign in to view

Copy link
@docentYT

docentYT Sep 27, 2023

Author Owner

Tak, tylko to nie ja tworzę tabelkę a po prostu znajduję tą co domyślnie ma librus na stronie i ona nie ma ID.
A jakbym miał dodawać ID przez rozszerzenie to i tak czy siak bym musiał użyć tego samego kodu aby ją najpierw znaleźć.

This comment has been minimized.

Sign in to view

Copy link
@EnterVPL

EnterVPL Sep 27, 2023

OK, da się to obejść, ale myślę że już nie warto. Musiałbyś sprawdzać też content i wszystkie pozostałe atrybuty elementów. To jest więcej roboty i też musiało by działać na szukaniu, takie AI, ale jednak nie. Póki co trzeba by to olać

let saturdayString = headers.childNodes[headers.childNodes.length-3].lastChild.textContent;
let saturday = new Date(saturdayString).setHours(0,0,0,0);
const saturdayString = headers.childNodes[headers.childNodes.length-3].lastChild.textContent;

This comment has been minimized.

Sign in to view

Copy link
@EnterVPL

EnterVPL Sep 27, 2023

dodaję punkt WTF, jako czytelnik kodu. Szczerze to nie wiem jak to naprawić, ale wygląda jakby miało się zepsuć
const saturday = new Date(saturdayString).setHours(0,0,0,0);

let sundayString = headers.childNodes[headers.childNodes.length-2].lastChild.textContent;
let sunday = new Date(sundayString).setHours(0,0,0,0);
const sundayString = headers.childNodes[headers.childNodes.length-2].lastChild.textContent;
const sunday = new Date(sundayString).setHours(0,0,0,0);

let today = new Date().setHours(0,0,0,0);
const today = new Date().setHours(0,0,0,0);

if (today == saturday || today == sunday) {
zmienTydzien(1); // function from original librus site
Expand Down

0 comments on commit 5d9f4f2

Please sign in to comment.