Dyskusja wikiprojektu:Sprzątanie kodu/Archiwum/4

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


Przypisy i pusta linia

Czy SK mogłoby w takiej edycji podmiany gołego references na szablon przypisy, robić też pustą linię? Bo tutaj wydaje mi się że to jest bardzo różne ten szablon przypisów i szablon nawigacyjny, że nie powinny być w jednej linii koło siebie. PMG (dyskusja) 16:03, 20 sty 2020 (CET)[odpowiedz]

2.89 Zrobione ~malarz pl PISZ 21:31, 28 lip 2020 (CEST)[odpowiedz]

SK usuwa spację w nazwie grafiki

[1], nazwa grafiki składa się z zakodowanych znaków w UTF. Wrzucam aby przy okazji postarać się znaleźć błąd. ~malarz pl PISZ 19:40, 28 lip 2020 (CEST)[odpowiedz]

U mnie OK. Beno @ 19:54, 28 lip 2020 (CEST)[odpowiedz]
Przed uruchomieniem zamień podkreślenie w nazwie pliku na spację. Problemem jest linia
str = str.replace(/([0-9]) (%|‰)/g, '$1$2');
zamieniłem na:
str = str.replace(/([0-9]) (%[^A-F0-9]|‰)/g, '$1$2');
2.88 Zrobione ~malarz pl PISZ 21:12, 28 lip 2020 (CEST)[odpowiedz]

Wyjątek do sprzątania kodu?

Któryś ze skryptów sprzątających kod w tej edycji popsuł wywołanie pliku, dodając spację przed rozszerzeniem. Przypuszczam że powodem jest cudzysłów znajdujący się bezpośrednio przed kropką przed rozszerzeniem. Ktoś może zlokalizować źródło problemu i dodać taką sytuację do wyjątków w skrypcie? Salicyna (dyskusja) 22:55, 3 paź 2020 (CEST)[odpowiedz]

Przenoszę z kawiarenki we właściwe miejsce. ~malarz pl PISZ 12:13, 4 paź 2020 (CEST)[odpowiedz]
Moja wina, pochylę się nad tym wieczorem. Beno @ 12:59, 4 paź 2020 (CEST)[odpowiedz]
Poprawiłem (diff). Beno @ 03:49, 6 paź 2020 (CEST)[odpowiedz]

Prostowanie monitora

Mam trochę problem z takim prostowaniem szablonu Monitora. Uważam że to zaciemnia ten szablon. Jaka jest wasza opinia? PMG (dyskusja) 11:11, 15 paź 2020 (CEST)[odpowiedz]

  • A nie masz problemu z "prostowaniem" np. szablonu Cytuj? Tam składnia jest 10 razy bardziej pokręcona. W Cytuj np. postaw źle średnik i problem gotowy, a intuicyjne nie jest tam to w żaden sposób i trzeba doktoratu, żeby używać. Tutaj masz naturalną kolejność i ładny, krótki kod. W czym problem? Użytkownik tego nie widzi. Beno @ 12:23, 15 paź 2020 (CEST)[odpowiedz]
    @Beno W szablonie {{Cytuj}} masz nazwane parametry przecież. To tylko w autorach masz jakieś dziwności, ale tego chyba mało kto używa (poza rozdzielaniem przecinkami). Natomiast jeśli to uważasz za skomplikowane, to szkoda, że nie wiesz jakie zasady narzucają sami sobie bibliotekarze i bibliotekarki. Znaki rozdzielające w formacie MARC to jest dopiero osobny kosmos 😉 Nux (dyskusja) 11:34, 20 paź 2020 (CEST)[odpowiedz]
Zgadzam się z @PMG. Nazwane parametry prawie zawsze są lepsze. Zwłaszcza w kodzie, a to jest sprzątanie kodu w końcu 😉. Dla VE to w sumie wszystko jedno, ale tutaj to chyba tylko prawnicy (i pewnie nie wszyscy) pamiętają czy "pozycja" jest przed czy po "numerze". Zamiana powinna być w drugą stronę (z numerowanych na nazwane). Nawiasem mówiąc - czy poprawiacz bierze pod uwagę, że parametry mogą być w nieoczekiwanej kolejności? Czy w takim wypadku skrypt też usunie nazwy parametrów: {{Monitor Polski|numer=261|rok=1936|pozycja=461}}? Ludzie nie powinni musieć pamiętać kolejności parametrów. Nux (dyskusja) 11:25, 20 paź 2020 (CEST)[odpowiedz]
  • Zrobiłem kwerendę i "oczekiwana kolejność" występuje prawie zawsze. Natomiast w przypadku "nieoczekiwanej kolejności" skrypt nic nie robi.
    Beno @ 22:06, 20 paź 2020 (CEST)[odpowiedz]
  • Ze skryptem są inne problemy:
    1. Zdarza się, że jest rok i pozycja, a nie ma numeru, wtedy nieistniejący numer wyświetla się z taką samą wartością jak pozycja:
      {{Dziennik Ustaw|rok=####|pozycja=####}}
      {{Monitor Polski|rok=####|pozycja=####}}
      insource:/Monitor *Polski *\| *rok *= *[0-9]+ *\| *p/ (kilka wystąpień)
      insource:/Dziennk *Ustaw *\| *rok *= *[0-9]+ *\| *p/ (kilkadziesiąt wystąpień).
      Jest to ewidentny błąd szablonu.
    2. Jest też problem składniowy natury redakcyjnej. Otóż dopuszczalny dla czytelnika jest zapis Rok+Pozycja, ale bez Numeru. Tak się praktykuje w części publikacji. Ale w składni naszych szablonów Monitora i Dziennika mamy coś takiego:
      {{Dziennik Ustaw|rok=####|numer=####}}
      {{Monitor Polski|rok=####|numer=####}}
      czyli parametr dotyczący pozycji nazywa się... „numer". Jest to mocno mylące.

      W zaistniałej sytuacji wypadałoby rozwiązać najpierw 1. problem, a następnie 2. i przebocić artykuły. Komuś pomyliły się nazwy obu parametrów - numeru i pozycji. Beno @ 22:06, 20 paź 2020 (CEST)[odpowiedz]

7B 7D

@Malarz pl W 2015 r. pojawił się wpis [2], a teraz w artykule Język norweski wp_sk zamienia

[[{{#invoke:Lang|hasło|non}}|{{#invoke:Lang|skrót|non}}]]
staronord.

na

[[.7B.7B#invoke:Lang|hasło|non}}|{{#invoke:Lang|skrót|non}}]]
hasło|non}}|staronord.

Beno @ 06:10, 1 gru 2020 (CET)[odpowiedz]

Nie odnosząc się do technikaliów - dlaczego mamy invoke w przestrzeni głównej? PMG (dyskusja) 07:44, 1 gru 2020 (CET)[odpowiedz]
Bo jakiś IPek przerabiał różne odwołania od języków. Trochę tego wyłapałem i poprawiłem, ale jak widać nie wszystko. IMO generalnie w PG nie powinno być żadnego {{#, bo jest to bez sensu i tylko utrudnia poprawianie kodu i edycje VE. ~malarz pl PISZ 10:01, 1 gru 2020 (CET)[odpowiedz]
Zgadzam się z tym podejściem. Tak jak nie puszczamy SK na szablonach, tak nie powinniśmy mieć w PG elementów które są przeznaczone dla szablonów. PMG (dyskusja) 12:46, 1 gru 2020 (CET)[odpowiedz]

Enter w szablonie {{Przypisy}}

Czy SK mógłby usuwać enter w szablonie {{Przypisy}} pomiędzy Przypisy a }}? Nie widzę sensu aby szablon zajmował dwie linijki, tutaj przykładowy diff. --The Polish (query) 12:54, 18 mar 2021 (CET)[odpowiedz]

Popieram tę prośbę. Mnie też wkurza takie wstawienie przypisów, więc fajnie jak by to SK sprzątało. PMG (dyskusja) 14:23, 19 mar 2021 (CET)[odpowiedz]
Zrobione diff. Przy okazji jeszcze parę innych wariantów, które gdzieś widywałem po edycjach VE. ~malarz pl PISZ 19:03, 19 mar 2021 (CET)[odpowiedz]

Zamiana łącznika na półpauzę

Któryś ze skryptów sprzątających kod popsuł wywołanie szablonu {{r}} w tej edycji. --The Polish (query) 01:41, 7 kwi 2021 (CEST)[odpowiedz]

Potraktowanie kreski pionowej w przypisach jako parametr infoboksu

Któryś ze skryptów sprzątających kod popsuł wywołanie przypisów w tej edycji. --The Polish (query) 02:54, 7 kwi 2021 (CEST)[odpowiedz]

  • To pewnie mój SI. Szczerze mówiąc to nawet nie wiedziałem, że w prostej składni LZ może być pionowa kreska. Takich linków jest dużo. Muszę się chwile zastanowić jak to naprawić. ~malarz pl PISZ 09:00, 7 kwi 2021 (CEST)[odpowiedz]

Zamiana apostrofu

Któryś ze skryptów sprzątających kod popsuł wywołanie szablonu {{listen}} w tej edycji (@Alan ffm). --The Polish (query) 21:40, 14 kwi 2021 (CEST)[odpowiedz]

@The Polish Spóźnione, ale załatwiłem. Beno (dyskusja) 02:09, 10 maj 2021 (CEST)[odpowiedz]

Propozycje zmian

Chciałbym zaproponować kilka dodatkowych zmian które wyłapałem podczas sprzątania WP:CHECK:

  • <br /> | parametr = | parametr = (parametr zakończony znacznikiem br)
    • analogicznie także <br /> }} }} (ostatni parametr szablonu
  • * XYZ <br /> * XYZ* XYZ * XYZ (znacznik br w liście)
  • ISBN:ISBN (różne warianty, może być spacja po dwukropku, przed dwukropkiem, może jej nie być - zmiana jest konieczna aby wyświetlał się magiczny link)
  • <nowiki>ISBN XYZ</nowiki>ISBN XYZ (niestety bardzo często VE wstawia ISBN pomiędzy nowiki - jest już to zgłoszone w Phabricatorze)
  • &laquo;«
  • &raquo;»
  • [[XYZ|„XYZ”]]„[[XYZ]]” (może też być “ ” lub " ")
  • [[XYZ|„XYZ”,]]„[[XYZ]]”, (może też być kropka)
  • „[[XYZ|XYZ”]]„[[XYZ]]”
  • [[XYZ|„XYZ]]”„[[XYZ]]”
  • [[XYZ|XYZ,]][[XYZ]], (może też być kropka)
  • [[XYZ|,XYZ]], [[XYZ]] (jak wyżej)
  • [[XYZ|(XYZ]]([[XYZ]]

The Polish (query) 05:21, 6 cze 2021 (CEST)[odpowiedz]

  • Wydaje mi się, że niektóre z nich są w skrypcie @Beno. ~malarz pl PISZ 11:44, 6 cze 2021 (CEST)[odpowiedz]
  • Coś mi się wydaje, że któryś z poprawiających, z komentarzami do edycji o używaniu któregoś z narzędzi, wstawiał znaki interpunkcyjne do ]]. Niektóre zgłaszający już wyczyścił i nie ma sensu dodawać ich do WP:CHECK. Moim zdaniem, to co jest na tyle oczywiste, że da się poprawić botem, i jest błędem, a nie doskonaleniem kodu, trzeba zebrać i zapuścić bota. Można coś na wzór strony z częstymi błędami zrobić wzorce do wyszukiwania w kodzie częstych błędów, chętni by co jakichś czas sprawdzali występowanie tych wzorców. Stok (dyskusja) 12:35, 6 cze 2021 (CEST)[odpowiedz]

  • <br /> | parametr = | parametr = (parametr zakończony znacznikiem br)
    • analogicznie także <br /> }} }} (ostatni parametr szablonu
      • Beno: wewnątrz szablonów cytowania usuwa podział wierszy oraz usuwa białe od wewnątrz znaków klamry
  • * XYZ <br /> * XYZ* XYZ * XYZ (znacznik br w liście)
    • Beno: usuwa breaka, zostawia odstęp, w długich listach czasami przydatne w celach redakcyjnych dla zaznaczenia jakiegoś większego podziału
  • ISBN:ISBN (różne warianty, może być spacja po dwukropku, przed dwukropkiem, może jej nie być - zmiana jest konieczna aby wyświetlał się magiczny link)
    • Beno: nie robi, nie znam problemu, u mnie nic się nie dzieje, jaka to sytuacja konkretnie?
  • <nowiki>ISBN XYZ</nowiki>ISBN XYZ (niestety bardzo często VE wstawia ISBN pomiędzy nowiki - jest już to zgłoszone w Phabricatorze)
    • Beno: nie robi, nie znam problemu, u mnie nic się nie dzieje, jaka to sytuacja konkretnie?
  • &laquo;«
    • Beno: robi
  • &raquo;»
    • Beno: robi
  • [[XYZ|„XYZ”]]„[[XYZ]]” (może też być “ ” lub " ")
    • Beno: nie robi, mam wątpliwości, czy to ma iść w tę stronę, ponieważ znaki cudzysłowu są integralną częścią wyrażenia, wg mnie powinno to iść w drugą stronę
  • [[XYZ|„XYZ”,]]„[[XYZ]]”, (może też być kropka)
    • Beno: przenosi przecinek
  • „[[XYZ|XYZ”]]„[[XYZ]]”
    • Beno: robi
  • [[XYZ|„XYZ]]”„[[XYZ]]”
    • Beno: robi
  • [[XYZ|XYZ,]][[XYZ]], (może też być kropka)
    • Beno: robi
  • [[XYZ|,XYZ]], [[XYZ]] (jak wyżej)
    • Beno: nie robi, 385 wystąpień (nie spodziewałem się aż tyle)
  • [[XYZ|(XYZ]]([[XYZ]]
    • Beno: ostrzega

Beno (dyskusja) 13:26, 6 cze 2021 (CEST)[odpowiedz]

  • Nie wiem jakim cudem nie miałem Twojego kodu włączonego, dlatego się zdziwiłem jak usłyszałem że Twój kod wszystko to robi. Już tłumacze. <nowiki>ISBN 978-83-900210-1-0</nowiki> daje efekt ISBN 978-83-900210-1-0, natomiast ISBN 978-83-900210-1-0 daje efekt ISBN 978-83-900210-1-0, stąd konieczność usunięcia nowiki, z nowiki magiczny link nie wyświetli się. Podobnie jest z dwukropkiem po ISBN, jeśli wstawimy dwukropek, to magiczny link nie zadziała. W artykule Budynek przy ul. Dybowskiej 15 w Toruniu Beno nie poprawia ani nie sygnalizuje [[Toruń|(Toruń]]). Ponadto widzę jeszcze że w artykule Steve Harris Beno nie poprawia [[Heavy metal|heavy metal.]] a w artykule Maria Celeste (telenowela) nie poprawia [[Pogarda (telenowela)|Pogarda (telenowela).]]. Oczywiście podobne przypadki będę przerabiał botem, ale aby się nie zdarzały, to proponowałbym wyłapywanie takich przypadków przez skrypt. The Polish (query) 14:46, 6 cze 2021 (CEST)[odpowiedz]
    • @The Polish 1) Acha, teraz rozumiem, chodzi o nowiki, 2) dzięki za wskazówki, rozbuduję skrypt. Beno (dyskusja) 22:16, 6 cze 2021 (CEST)[odpowiedz]
    • Z tym ISBN-em dzieje się tak , bo wszystko, co znajdzie się wewnątrz znacznika <code><nowiki></code>, oprogramowanie traktuje jako rawtext. XaxeLoled AmA 23:34, 6 cze 2021 (CEST)[odpowiedz]

Błąd wewnątrz szablonu Szablon:Populacja w latach

W tej edycji popsuło się wyświetlanie wykresu liczby mieszkańców. Mario58 -- Skrobnij zdanko 09:07, 18 cze 2021 (CEST)[odpowiedz]

  • To mój szablon. Pierwsze próby użycia nowych rozszerzeń do tworzenia wykresów. Chyba wymaga danych w postaci JSON. Wydawało mi się, że przekazywanie tych danych w gołej postaci będzie najprostszym rozwiązaniem. Jak widać myliłem się. W tle działają przecież różne rozszerzenia. Paweł Ziemian (dyskusja) 18:49, 18 cze 2021 (CEST)[odpowiedz]
    • @Paweł Ziemian Nic się nie zmieniło. Goły wp_sk jest OK, z moim błąd jest. Szukałem u mnie, ale nie znalazłem. U mnie błąd jest po użyciu skryptów [[WP:SK]]+[[WP:SK/mSI|mSI.v2]]+[[WP:SK/Bn|Bn]]. Nie wiem, czy dalej szukać u siebie? Beno (dyskusja) 09:45, 20 cze 2021 (CEST)[odpowiedz]

Niesparowane nawiasy okrągłe

Nie wiem czemu SK dodaję komunikat o niesparowanych nawiasach w tej edycji. Wszystkie nawiasy mają swój odpowiednik. The Polish (query) 11:39, 23 cze 2021 (CEST)[odpowiedz]

@The Polish Prawy nawias to egzotyczny wynalazek pod nazwą FULLWIDTH RIGHT PARENTHESIS (U+FF09), a powinien być zwykły, chamski, czyli RIGHT PARENTHESIS (U+0029). Nie widzę sensu stosowania tych wymyślnych znaków, lekko tylko odbiegających wyglądem od normalnych. Zaraz wpiszę regułkę przemieniającą oba brackety (znaczy lewy też). Beno (dyskusja) 14:30, 23 cze 2021 (CEST)[odpowiedz]

Dublowanie się przypisów bezpośrednio po sobie.

W pierwszej wersji hasła autor wstawił dwa przypisy jeden za drugim. Czy można wzbogacić któryś ze skryptów SK, żeby usuwać zdublowane, sąsiadujące przypisy? Nie jest to pierwszy przypadek, jaki widzę w tym tygodniu. Mario58 -- Skrobnij zdanko 19:54, 18 cze 2021 (CEST)[odpowiedz]

Przypadek troszkę trudniejszy. Zdublowane przypisy były rozdzielone innym. SKrypt w ogóle nie ruszył zdublowanego przypisu, usunąłem go ręcznie. Mario58 -- Skrobnij zdanko 08:55, 20 cze 2021 (CEST)[odpowiedz]
Kolejny przypadek zdublowanych kolejnych przypisów. Mario58 -- Skrobnij zdanko 16:51, 5 lip 2021 (CEST)[odpowiedz]

Błąd przy wywołaniu #invoke

W haśle Czarnocin chciałem posprzątać kod, ale skrypt nie poradził sobie z fragmentem ([[Dwujęzyczne nazewnictwo geograficzne w Polsce|dodatkowa nazwa]] w [[{{#invoke:Lang|hasło|de}}|j. {{#invoke:Lang|skrót|de}}]]. Pierwsze nawiasy klamrowe zamienił na .7B.7B.. Zgłoszono już podobny przypadek. Mario58 -- Skrobnij zdanko 17:31, 16 lip 2021 (CEST)[odpowiedz]

Tego w ogóle nie powinno być w artykule. ~malarz pl PISZ 21:32, 16 lip 2021 (CEST)[odpowiedz]

Błąd w uwagach w artykułach wojskowych

Zapewne któreś z rozszerzeń w uwagach w artykułach wojskowych jest nadgorliwe. Poprawia zdania typu W 1920 dowodził kompanią 74 pp. 1 grudnia 1924 został mianowany kapitanem ze starszeństwem z 1 lipca 1923 i 99,5. lokatą w korpusie oficerów piechoty na W 1920 dowodził kompanią 74 s. 1 grudnia 1924 został mianowany kapitanem ze starszeństwem z 1 lipca 1923 i 99,5. lokatą w korpusie oficerów piechoty. Z kontekstu wynika, że pp oznacza tutaj pułk piechoty, a nie strony, czego narzędzia WP:SK nie ogarniają. Mario58 -- Skrobnij zdanko 09:59, 26 sie 2021 (CEST)[odpowiedz]

Nie wrzuciłem diffa. Oto i on: Komenda Uzupełnień Rejonu Nisko. Mario58 -- Skrobnij zdanko 10:01, 26 sie 2021 (CEST)[odpowiedz]
@Mario58 To robiło moje rozszerzenie. Wyłączyłem. Beno (dyskusja) 17:35, 26 sie 2021 (CEST)[odpowiedz]