Filter om erotiek effectiever te weren

[Megalos]

Dat klopt inderdaad niet, maar zo had ik hem ook niet (zal wel een typfoutje hebben gemaakt in mijn post ). Ik had hem zo:

Code: Selecteer alles

if (title.substring(0, 6) != '(Spam)' && group.substring(13, 20) != ('erotica' || 'ijsklon')) {
	remove=0
} else {
	remove=1

Ik zal jouw suggestie eens proberen. Maar wat doet dat group.length > 19 eigenlijk? Waarom is dat nodig?

[Ep Woody]

Dat zorgt ervoor dat die controle op erotica alleen gedaan wordt als de groupnaam langer is dan 19 karakters. Op die manier kan hij die test niet doen als de group leeg is. Daar meot je even mee spelen.

wat ook kan is een extra if ervoor if (group.length>5) . Dan doet hij die hele controle pas als de group.length gevuld is.

[Megalos]

Met dat extra if statement werkt het perfect nu! Bedankt voor de reacties. Toch weer een extra beveiliging voor de mensen die dit willen gebruiken.

Update: Mm, iets te vroeg gejuicht. Hij geeft nu bij alle spots een spam melding. Ik heb nu het volgende:

Code: Selecteer alles

<script>
badwords('commentsbody'); 
//badwords('description');

title = "{title}"
group = "{newsgroup}"

if (group.length > 5) {
if (title.substring(0, 6) == '(Spam)' || group.substring(13, 20) == ('erotica' || 'ijsklon'))) {
   remove=1
} else {
   remove=0
}
if (remove == 1) {
	document.getElementById('spamwarning').style.display = "block"; 
	document.getElementById('bodybox').style.display = "none";
} else {
	document.getElementById('spamwarning').style.display = "none"; 
	document.getElementById('bodybox').style.display = "block";
}}

</script>

[Wheaten]

ten eerste, kloppen je haakjes niet: (meer sluitende als opende haakjes)
(title.substring(0, 6) == '(Spam)' || group.substring(13, 20) == ('erotica' || 'ijsklon')))

Vervang code door:

Code: Selecteer alles

<script>
badwords('commentsbody'); 
//badwords('description');

title = "{title}"
group = "{newsgroup}"

if (group.length > 5) {
     if ((title.substring(0, 6) == '(Spam)') || (group.substring(13, 20) == ('erotica' || 'ijsklon'))) {
        document.getElementById('spamwarning').style.display = "block"; 
        document.getElementById('bodybox').style.display = "none";
     }
    else {
     document.getElementById('spamwarning').style.display = "none"; 
     document.getElementById('bodybox').style.display = "block";
    }
}
</script>

[Megalos]

Bedankt Wheaten, nu lijkt het prima te werken.

[Wheaten]

Graag gedaan

[hjvbft]

Mooi dat het gelukt is. Nu kunnen andere dit ook gebruiken als ze dit willen.
Is het hiermee nog mogelijk om de "erotica" spots te openen na tonen waarschuwing of kunnen de spots niet geopend worden?

De 2 woorden waarop nu gecontroleerd word zorgt ervoor dat het grootste deel erotiek spots herkend word.
Om nog meer spots als erotiek te herkennen kan de lijst met woorden nog verder uitgebreid worden.
Met bovenstaand voorbeeld kunnen op dezelfde manier meer woorden toegevoegd worden die specifiek voor erotiek gebruikt worden.

[Megalos]

Het is nog wel mogelijk om ze te openen na de waarschuwing. Net zoals dat het geval is bij overige Spam berichten. Ideaal zou natuurlijk zijn als er ook een mogelijkheid komt waardoor dat niet meer kan. Nog beter: dat ze niet meer in het overzicht komen te staan. Maar dat was nog niet zo makkelijk om te implementeren had ik vernomen.

[Wheaten]

wat je kunt doen is een aanroep naar de spots.db te doen en de code in je if statement als delete instructie uit te voeren

[Megalos]

Dat zou mooi zijn inderdaad. Dat je bijvoorbeeld kunt aangeven of de spots echt verwijderd moeten worden (of bijvoorbeeld alleen maar als 'hidden' bestempeld moeten worden). Heeft iemand een idee wat daarvoor de code is, ikzelf ben echt een leek hierin.