
Wpis użytkownika bartek555 w Hydepark
bartek555Lider
94piorunówno skislem xD nie bedziesz robaku wyjezdzac poza europe!
Komentarze (33)
@bartek555 mnie rozwala, bo https://rozklad-pkp.pl/ (portal na którym można znaleźć i kupić bilet na pociąg) często nie działa za granicą, jak ktoś wraca do kraju i ląduje w Warszawie, to nie może sobie kupić biletu na ciapong do swojego miasta tam. xD
Poprawiony kod przez chat GPT, który usuwa bugi: https://pastebin.com/6qvNCnaq
(function ($) { "use strict"; $.fn.zakupBiletu = function (options) - Pastebin.comPastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.Pastebin@tomasz-frankowski
Important bugs I found
A few things in the original are particularly suspicious:
1. dalej_modal_false = true means this branch is effectively always selected:
} else if (dalej_modal_false) {
modalService.dalejModalFalse();
}
Consequently, dalejModalDefault() is effectively unreachable.
2. These conditions are always truthy:
if (strefaCiszyArr && ...)
An array exists, so strefaCiszyArr itself doesn't test anything.
3. String/number comparison:
if (liczba_n === 0 && liczba_u > 0 ...)
$("#liczba_n").val() returns a string. This should use numeric conversion.
4. This appears to be a selector bug:
znizkaWrapper.find("liczba_u2" + (i + 1))
It is missing #.
5. This comparison can never be useful:
var klasa_poc1 = $("#klasa_wagonu").find("option:selected").val();
var klasa_poc2 = $("#klasa_wagonu").find("option:selected").val();
if (klasa_poc1 !== klasa_poc2) {
Both values come from exactly the same element.
6. This:
.prop("disabled", "disabled")
should be:
.prop("disabled", true)
7. There are many places where the code repeatedly does:
$("#...")
$("....")
and repeatedly reads the same values. This makes the main event handler much harder to reason about.
8. There are several unused variables, including:
rower
v3
v4
v5
v6
arrow_box_ext_info // depending on later usage
Some of those may have historical reasons, so I'd remove them only after checking the rest of the application.
@tomasz-frankowski is_pies?
@Deykun przypomina mi to złoto z kodu źródłowego PKP xD
https://bilet.intercity.pl/eic_js/zakup_biletu_plugin.js?ver=880871484
@bartek555 musisz odpalić VPN, nie masz wyjścia.
@DexterFromLab może ma VPNa od jednego z tych chińskich dostawców co udają zachodnie firmy 😛
@bartek555 blokują serwery VPN bo próbują je namierzać. A Twój dostawca VPN zmienia serwery na inne. Zabawa w kotka i myszkę. Tak już będzie, raz zadziała a raz nie.
@Aksal89 w sumie to zaczal dzialac znowu, nie wiem co tam sie odkurwia
@Aksal89 no to elo, to pulpit zdalny z komputera z domu xD
@DexterFromLab pisał wczoraj, że przez VPN już nie działa, chociaż działało wcześniej
rownam poziom
@bartek555 teraz to bedzie "ej, pobite gary"
@bartek555 Haha, daj znać co odpiszą :grinning:
@bartek555 czyli jednak płacisz. Zamykam temat.
@bartek555 Wiesz, że to nie tak działa :confused: Zaraz Ci odpiszą, żebyś sobie ustawił płatności automatyczne xD
@bartek555 tak samo w poprzedniej firmie wiesz po co prezes trzymał ochroniarza 24/7?
Cytuję:
@bartek555 bo to chodzi o to, żeby w raporcie wpisać, że wprowadzono zabezpieczenia
po co ci prąd jak cię w domu nie ma
Opłata mocowa
Opłata jakościowa
Opłata przejściowa
Opłata za opłatę
Opłata płata łata opata
Opłata bo możemy
Opłata jeszcze złotówka nie zaszkodzi
@bartek555 no i nie będziesz miał od nich prądu w Azji. Szach mar!
@Felonious_Gru @tomasz-frankowski swoja droga - zajebiste zabezpieczenie, jesli vpn wystarczy, zeby je obejsc xD hakierzy nie dadza rady
@Opornik włamuj się do wlasnego dostawcy żeby mu w zębach pieniądze przynieść xd
@bartek555 angry enea caveman noises
@Opornik ale to nie o to chodzi żeby obchodzić, tylko żeby mieć możliwość otrzymania i zapoznania się z fakturą
@evilonep bo przecież nie ma vpn-ów i innych gówien żeby to obejść
@bartek555 po co ci prąd jak cię w domu nie ma
@bartek555 ano powinni, Energa wysyła
@evilonep w takim razie niech wysylaja efakture na maila
@UncleFester nie maja za co, bo place 😉
@bartek555 Woda na ich młyn, dowalą odsetki. :smiling_imp:
@bartek555 najprawdopodobniej wynika to z unijnej dyrektywy NIS2 energetyka to sektor krytyczny więc blokują taki ruch 😛
@bartek555 pewnie im się po tym styki przepaliły.
@bartek555 makao i po makale ( ͡~ ͜ʖ ͡°)