Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blokowanie wysłania zgłoszenia dla artykułów poniżej limitu 2kB #5

Open
czupirek opened this issue Jan 12, 2024 · 5 comments
Open
Labels
enhancement New feature or request feedback required Needs more information from community help wanted Extra attention is needed

Comments

@czupirek
Copy link

Potrzebne blokowanie, gdy artykuł ma <2kB - tak, jak działa to dla artykułów bez źródeł. Tzn. oprócz tego czerwonego napisu.

@Eccenux
Copy link
Owner

Eccenux commented Jan 12, 2024

Jest komunikat. Wystarczy.

@Eccenux Eccenux closed this as completed Jan 12, 2024
@Eccenux Eccenux reopened this Jan 13, 2024
@Eccenux Eccenux added enhancement New feature or request help wanted Extra attention is needed and removed wontfix labels Jan 13, 2024
@Eccenux
Copy link
Owner

Eccenux commented Jan 13, 2024

Zrobiłem już udoskonaloną wersję ustalania wielkości artykułu i ostatnich zmian w artykule. Nadal trzeba brać pod uwagę, że artykuł mógł zostać przepisany. Przepisany artykuł mający nominalnie tyle samo bajtów może mieć 10kB nowych danych...

Ale można by pokombinować kiedy faktycznie zgłoszenie powinno być blokowane.

@Eccenux Eccenux changed the title Blokowanie wysłania zgłoszenia dla < 2kB Blokowanie wysłania zgłoszenia dla artykułów poniżej limitu 2kB Jan 13, 2024
@Eccenux Eccenux added the good first issue Good for newcomers label Jan 13, 2024
@Kaligula0
Copy link
Contributor

Kaligula0 commented Jan 30, 2024

Co robi ta udoskonalona wersja? Bo moim marzeniem zawsze był pomiar ilości samego tekstu, tj. bez kategorii, szablonów, (przypisów?) itp., ale nie udało mi się tego zrealizować.

@Eccenux
Copy link
Owner

Eccenux commented Jan 31, 2024

Co robi ta udoskonalona wersja? Bo moim marzeniem zawsze był pomiar ilości samego tekstu, tj. bez kategorii, szablonów, (przypisów?) itp., ale nie udało mi się tego zrealizować.

Tego nie robiłem. Zresztą nie jestem pewien jakie jest tutaj oczekiwanie ludzi powiązanych z projektem CW. Jamnik faktycznie coś wspomniał, że w zasadzie infoboks się nie do końca liczy w CW, ale z drugiej strony w konkursach liczy się każdy bajt (w tym przypisy)... Tak że to byłoby pewnie do ustalenia i to raczej nie byłoby łatwe.

W każdym razie RevisionList umie liczyć różne przypadki. Dzięki dokładnemu sprawdzeniu historii zmian można ustalić:

  • czy artykuł jest nowy,
  • jak dużo było w sumie zmian i jaka jest wielkość artykułu.

Czyli można w miarę łatwo dodać twardy limit dla nowych artykułów (tylko dla nowych, bo przy przebudowie można mieć 0 bajtów różnicy, a przepisać cały artykuł).

@Eccenux
Copy link
Owner

Eccenux commented Jan 31, 2024

Metoda liczenia długości tekstu

Chyba wymyśliłem jak można by uczciwie liczyć ilość tekstu na stronie. Można ukryć zbędne szablony i linki edycji:

$('.ambox,.navbox,.mw-editsection').hide();

I potem sprawdzić jaka jest długość:

visibleText = $('#mw-content-text')[0].innerText
textChars = visibleText.length;
textBytes = (new TextEncoder().encode(visibleText)).length;
console.log(`długość: ${textChars} znaków`);
console.log(`długość: ${textBytes} bajtów`);

To sobie zapisać i jeśli artykuł jest nowy, to według tego liczyć limit.

Wyświetlanie długości tekstu

Powyższe wyliczenie należałoby też dorzucić w sekcji z tabelką zmian (może nad tabelką):

Wielkość w momencie zgłoszenia: ${textChars} znaków.

obraz

PS: Nawiasem mówiąc znaki to nie bajty... A w sumie to ciekawe, jak mediawiki liczy np. "żółć" - jako 4 (znaki) czy 8 (bajtów)? Trzeba by sprawdzić tak dla formalności. MW liczy bajty (pokazuje bajty-bajty w historii zmian).

PPS: Dla spornego artykułu wychodzi:

  • długość: 2323 znaków
  • długość: 2440 bajtów

Tak że na pewno w polskich artykułach różnica jest znacząca.

@Eccenux Eccenux added feedback required Needs more information from community and removed good first issue Good for newcomers labels Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feedback required Needs more information from community help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants