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

Do not allow script tags in Koha news - ksdev/ks-0037-KD-4348 #25

Open
AnneliO opened this issue Sep 20, 2023 · 3 comments
Open

Do not allow script tags in Koha news - ksdev/ks-0037-KD-4348 #25

AnneliO opened this issue Sep 20, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@AnneliO
Copy link
Contributor

AnneliO commented Sep 20, 2023

Siirrettävä ominaisuus

Do not allow script tags in Koha news.

@AnneliO AnneliO added the bug Something isn't working label Sep 20, 2023
@AnneliO
Copy link
Contributor Author

AnneliO commented Nov 20, 2023

Koha-Suomen asiantuntijaryhmä 20.11.2023: Kodo tutkii, tarvitaanko. Jos tarvii, niin siirretään.

@CodoDerDritte
Copy link

CodoDerDritte commented Aug 2, 2024

Tämä näyttää nyt vähän kaksipiippuiselta. Ensin hyvät uutiset:

  • Kaikkia skriptitägejä ei enää ajeta joka kerta kun uutiset näytetään, eli siinä mielessä ongelma on poistunut.
  • Jos uutisessa on tekstiä, ei skriptitägiä myöskään ajeta "Additional content: news" sivulla, myös siinä mielessä ongelma on poistunut.

Ja sitten huonot:

  • Skripti-tägit ajetaan Additional-content sivulla jos uutinen muuten on tyhjä (eli ei sisällä tekstiä), tässä mielessä ongelma on edelleen olemassa.

Vakavin ongelma tuosta (eli skriptitägien ajaminen joka kerta kun uutiset näytetään esimerkiksi Kohan etusivulla) on poistunut, joten ehkä voimme elää tämän puutteen kanssa. Nähdäkseni tämä ei enää mahdollista semmoisia tuhmuuksia kuin aiemmin. Tämän voisi ehkä raportoida yhteisöön bugina ja ehkä korjata yhteisön kautta.

@CodoDerDritte
Copy link

Komento takas, kyllä se sittenkin ajetaan myös etusivulla. Pah.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: No status
Status: Työn alla
Development

No branches or pull requests

2 participants