Dyskusja wikiprojektu:Sprzątanie kodu

Z Wikipedii, wolnej encyklopedii
Przejdź do nawigacji Przejdź do wyszukiwania

param = }} - czy taka spacja (między = a }) nie mogłaby pozostawać?[edytuj kod]

Czy ta spacja pomiędzy = } nie mogłaby pozostać?

diff, Wikipedysta:Nux/wp_sk.js#L-407. MarMi wiki (dyskusja) 23:42, 12 gru 2021 (CET)[odpowiedz]

Mój skrypt też usuwa. I zapewne wiele innych. To jest bardzo dobre rozwiązanie, bo masz pod ręką spójny koniec wyrażenia. Gdyby była spacja, nie miałbyś nad tym kontroli, a w przypadku odpowiedniej szerokości okna ta klamra lądowałaby samotnie w nowym wierszu i nie wiesz, czy to przypadek, czy wbity enter. Natomiast co do wypełniania, to nie ma sprawy, zawsze na koniec i tak klikamy powtórnie w przycisk sprzątania kodu, który wszystko porządkuje. Beno (dyskusja) 02:29, 13 gru 2021 (CET)[odpowiedz]

Samozamykający się tag nowiki[edytuj kod]

Ale samozamykajacy się tag nowiki (<nowiki />) to chyba SK powinno już inaczej traktować niż #Niedomknięty tag nowiki (Iron Maiden)? MarMi wiki (dyskusja) 01:33, 15 gru 2021 (CET)[odpowiedz]

Po wejściu na wersję hasła podaną przez MarMi i puszczeniu tam SK ciąg znaków

„[[The Number of the Beast (singel)|The Number of the Beast]]”

jest zamieniany na

[[The Number of the Beast (singel)|The Number of the Beas]]<nowiki />t”

Ja nie mam rozszerzenia Beno. PMG (dyskusja) 09:59, 15 gru 2021 (CET)[odpowiedz]

  • @PMG Te nowiki tam było już w tej edycji a SK nie zmienia go, porównując z bieżącą (a "podgląd zmian" zawsze porównuje z bieżącą) masz efekt "dodania". @Nux zajrzysz do funkcji wp_sk.nowiki.hide aby traktowała odpowiednio samozamykające tagi. @MarMi wiki poza tym, że rzeczywiście ten tag jest źle traktowany to i tak nie powinno go być w artykule. ~malarz pl PISZ 13:35, 15 gru 2021 (CET)[odpowiedz]
    U mnie wp:sk nic nie zmienia test. Zresztą byłoby to dziwne jakby zmieniał. Trudno mi sobie wyobrazić jakiś przykład realnego zastosowania nowiki po linku jak właśnie celowo wyłączenie "t" z linka. Czyli jak dla mnie działa OK. Nux (dyskusja) 17:14, 15 gru 2021 (CET)[odpowiedz]
    Problem jest taki, że w obecnym kształcie Wikipedysta:Nux/test link nowiki WP:SK działa tylko do nowiki, całkowicie pomijając późniejsze elementy. Obecna funkcja wp_sk.nowiki.hide ukrywa wszystko od nowiki do końca i funkcja cleanerWikiVaria operuje jedynie na fragmencie od początku do nowiki. Wykonanie WP:SK na tym dokumencie poprawi pierwszy nagłówek i nie zrobi tego samego z trzecim, choć IMO nie ma żadnych przeciwwskazań. ~malarz pl PISZ 20:25, 15 gru 2021 (CET)[odpowiedz]
    A, faktycznie. No to bug. Nux (dyskusja) 20:55, 15 gru 2021 (CET)[odpowiedz]
    A dokładniej nie poprawi drugiego, który jest pomiędzy < nowiki/> a parą nowiki, bo
    wp_sk.nowiki.hide nie uwzględnia samozamykających się tagów, dopasowując do < nowiki/> najbliższy zamykający </ nowiki>.
    O ile jeszcze przy jedynym niedomkniętym tagu ucięcie do końca dostępnej treści do sprzątania jeszcze jest jakoś usprawiedliwione (bo może trudno określić w którym miejscu nowiki będzie automatycznie zamykany), to samozamykający się tag ma raczej krótki zasięg. MarMi wiki (dyskusja) 03:13, 16 gru 2021 (CET)[odpowiedz]
OK, nowa wersja weszła do gadżetu. Tagi `nowiki/` i `pre/` powinny być prawidłowo ignorowane (nawet jeśli są samo-zamknięte). Nux (dyskusja) 02:26, 18 gru 2021 (CET)[odpowiedz]
@Nux A czy tag samozamykający nowiki jest w ogóle do czegoś potrzebny? Czy nie lepiej je po prostu usuwać? Beno (dyskusja) 17:07, 19 gru 2021 (CET)[odpowiedz]
Ma swoje zastosowania (~2890), choć część pewnie można by zrobić w inny sposób. Usuwanie wszystkiego z automatu nie jest wskazane. Ale co poniektóre przypadki można pewnie przerobić/pousuwać automatycznie - np. po wikilinkach (wyjęcie końcówek z linkowania, to raczej do niczego nie jest potrzebne?). MarMi wiki (dyskusja) 17:44, 19 gru 2021 (CET)[odpowiedz]
Przyjrzę się temu i posprzątam te nowiki które są w końcówkach linkowań i takich tam. Mam wrażenie że to nowiki w tych poprzerywanych linkach jest jakoś wstawiane przez VE ale nie potrafię tego zreprodukować. PMG (dyskusja) 21:35, 19 gru 2021 (CET)[odpowiedz]
przykład takiego linkowania że chyba nikt kto widzi kod by tak nie zrobił. PMG (dyskusja) 22:02, 19 gru 2021 (CET)[odpowiedz]
czasem jednak widać że ktoś chciał bardzo coś zrobić, ale nie wiedział jak. Będę przy tym siedział. PMG (dyskusja) 22:17, 19 gru 2021 (CET)[odpowiedz]
  • Update: posprzątałem tego dziś jakieś 300. Idzie trochę szybko, trochę wolno. Mam popisane pomagacze, więc samo się klika w większości, ale zawsze i tak trzeba sprawdzać okiem - więc na pewno nie można na to bota puścić. Jakieś 10% haseł trzeba mocno zmieniać bo czasem nie wiadomo co użytkownik chciał osiągnąć (albo co mu się przekleiło). Ogólnie - na pewno nie na bota, mi to też trochę czasu zajmie. Ale raczej tygodnie niż miesiące. PMG (dyskusja) 20:09, 20 gru 2021 (CET)[odpowiedz]
  • update - daje mi to zadanie trochę w kość, bo muszę robić takie edycje i trochę mnie to boli. Ale jakoś idzie. Obecnie ostatnie 20 stron z listy to już same takie "tego nie będę zmieniał" (link tutaj, ale będę sprzątał kolejne rzeczy wiec będzie za chwilę nieaktualny. Ale ogólnie - ostatnie 20 haseł).
    • @PMG Nie rozumiem, po co ingerujesz w trzy prawe nawiasy kwadratowe. Przecież one wyświetlają się poprawnie. Nie trzeba tam niczego dodawać. Problem jest tylko z trzema lewymi - otwierającymi. Usunąłem je tutaj link i nie widzę różnicy. Beno (dyskusja) 14:48, 10 sty 2022 (CET)[odpowiedz]
    Bo kod ma nie tylko działać ale też być czytelny. To że strona działa po twojej edycji to jedno. To że kod ma być logiczny i jak nowikujesz otwarcie to przydałoby się też nowikowac zamknięcie żeby kolejni ludzie edytujący to hasło wiedzieli dlaczego tak a nie inaczej. Jak bym miał pomysł jak to zmienić w tekście to bym wywalił całkowicie te otaczające kwadratowe nawiasy. Niestety tuż obok jest już użyty zwykły nawias, więc nie wiem na co to zamienić. PMG (dyskusja) 15:54, 10 sty 2022 (CET)[odpowiedz]
    @PMG Wstawianie tagów, które nic nie robią, kłóci się z moim poczuciem logiki. A zrobić można jeszcze inaczej, np.:
    [<nowiki />[[Abc]]] <- samozamykający w poprawnej składni ze spacją
    &#91;[[Abc]]]
    Beno (dyskusja) 17:48, 10 sty 2022 (CET)[odpowiedz]

Podsumowanie[edytuj kod]

@MarMi wiki, @malarz pl, @Nux Skończyłem ręczne zmiany. Widziałem rzeczy których chciałbym nie widzieć, takie jak

Wiki<nowiki/>[[Wikipedia|ped]]ia

Ale skończyłem. Na pewno nie dało się tego sprzątnąć botem, skoro dla człowieka często było trudno. Co zostało? kilka grup:

  • w infoboksach chemicznych cuda związane z nazwami (przykład: Metotreksat)
  • wszelkie wystąpienia "chcę mieć ładnie wypunktowane rzeczy, a pierwszą gwiazdkę zawsze mi niszczy" (przykład: Mars)
  • przypisy do szematyzmu (przykład: Józef Białynia Chołodecki)
  • specjalna metasytuacja w Skala Celsjusza

Rzeczy które pozostały są tutaj, ale już ich ruszać nie zamierzam. Zmęczyła mnie mocno walka z tymi edycjami. Pod koniec to był taki trochę marsz na zmęczenie żeby tylko skończyć. PMG (dyskusja) 13:33, 24 sty 2022 (CET)[odpowiedz]

Uwagi do listy:
  • Metotreksat i inne - można użyć zamiennika z {{!}}, tylko czy warto? Skoro obecnie SK obsługuje samozamykające nowiki.
  • Mars - to zdaje się poprawne użycie nowiki, kiedyś widziałem w opisie do któregoś szablonu żeby tak robić przy wypunktowaniach (inna sprawa, czy to musi być wypunktowane...)
  • szematyzm - nowiki w przypisach zupełnie niepotrzebne, skoro wypunktowania i tak są w nowej linii (jak się usunie nowiki nie robi to żadnej różnicy)
  • C - tu nie wiem o co chodzi, bo gadżet SK tego w ogóle nie rusza (obecnie) - być może robi to któreś z rozszerzeń. MarMi wiki (dyskusja) 19:06, 24 sty 2022 (CET)[odpowiedz]
Rusza, rusza (w cleanerTXT jest sprzątanie stopni Celsjusza). Zresztą niestety SK robi to w zasadzie w złą stronę. Chyba Beno chciał poprawić, ale okazało się, że Wikipedyści się przyzwyczaili do własnego uzusu stworzonego przeze mnie kiedyś chyba 🙈😅... Nux (dyskusja) 21:16, 24 sty 2022 (CET)[odpowiedz]
A faktycznie, SK dodaje spację przed kółeczkiem. Ale pozostałe nowiki były zbędne (przynajmniej przy SK bez rozszerzeń).
Edycja: A Wikipedysta:Beno/wp_sk.js nawet stopniom w komentarzu nie daje spokoju. MarMi wiki (dyskusja) 23:06, 24 sty 2022 (CET)[odpowiedz]
Artykuł o stopniach Celsjusza ma przykłady różnych rodzajów pisowni, dlatego jest nietypowy, ale tak go przerobiłem, że wszystko jest ok i skrypt niczego nie zmienia. Spacja przed kółeczkiem jest poprawna, bo tak się pisze jednostki miary. Beno (dyskusja) 01:25, 25 sty 2022 (CET)[odpowiedz]
A, już pamiętam... to jednak Julo chciał poprawić za RJP [3]. No w każdym razie jak jest nowiki, to SK nie będzie ruszać tego w żadną stronę. Nux (dyskusja) 00:25, 26 sty 2022 (CET)[odpowiedz]
Pousuwałem dalsze nowiki (z przypisów i z tych szablonów, gdzie się to dało zrobić), większość co została to głównie artykuły chemiczne i listy w parametrze szablonu.
Nie wiem czy jest sens ruszać nowiki w chemii.
VE czasami wstawia nowiki (albo raczej Funkcja sugerowania linków). MarMi wiki (dyskusja) 23:24, 24 sty 2022 (CET)[odpowiedz]

Sprzątnięcie URL[edytuj kod]

Któreś z SK sprzątnęło w tej edycji (szukajcie "Scrapped_Minilla_heat_ray") URL, co rozbiło szablon. Dałoby się to jakoś powstrzymać? PMG (dyskusja) 11:32, 19 sty 2022 (CET)[odpowiedz]

  • Mój zestaw nic nie zepsuł, więc drogą eliminacji wychodzi, że to wina rozszerzenia ToStera. Sugeruję rezygnację z niego. ~malarz pl PISZ 12:40, 19 sty 2022 (CET)[odpowiedz]

Egipskie hieroglify[edytuj kod]

Mam kłopot z uruchomieniem sprzątania kodu w Egipskie hieroglify – określniki i ideogramy i Lista egipskich hieroglifów (gdy jest otwarty do edycji cały artykuł). Podejrzewam związek z tym, że są to jedne z najdłuższych artykułów, z bardzo rozbudowanym kodem, licznymi tabelami, licznymi znacznikami HTML, znacznikami wikikodu i znacznikami rozszerzenia WikiHiero; być może, że to problem o charakterze wydajnościowym. Korzystam z WP:SK+Bn --WTM (dyskusja) 21:49, 24 sty 2022 (CET)[odpowiedz]

Za dużo rekursji w 2. (Egipskie hieroglify) i 3. (Lista) wierszu z ... 7 identycznych (jeden nie wystarcza?):
str = str.replace(/<([^<>]+)([^<>]*)>([\s\S]*?)<(\/\1)>/g, '\x01$1$2\x02$3\x01$4\x02'); MarMi wiki (dyskusja) 23:47, 24 sty 2022 (CET)[odpowiedz]

Wyrzucenie informacji o języku polskim w szablonach cytowań[edytuj kod]

Jakoś nie jestem przekonany do tej zmiany. Zawsze to mniej informacji, które mogą się kiedyś, gdzieś przydać np. gdyby komuś przyszło tłumaczyć półautomatycznie szablony na inne języki. Wtedy nie wiadomo, czy język nie został podany, czy faktycznie jest to j. polski. Emptywords (dyskusja) 08:26, 14 maj 2022 (CEST)[odpowiedz]