Keyword highlighting BUG

[Marcel van Veen]

http://tweakers.net/nieuws/15550?&mode=thread&ParentID=258638&niv=-1&highlight=IBM%2BUltrastar%2BCheetah

Ga daar maar eens kijken, en dan vooral naar de eerste reactie van Femme. Try the link, en je zult zien dat die helemaal kaputt is

Foutje in de highlighting code?

[Femme]

Heeft er iemand een regex in de aanbieding die dat oplost? (ik heb iets nodig dat alleen buiten < > tags werkt)

Ik heb zelf al wel wat aan zitten klooien maar ik krijg niet iets dat goed werkt (/me doet op reguliere expressies).

[Onno]

$test = "<blaat> blaat <b> blaat </b> </blaat>\n";

$test =~ s/(>[^<]*)(blaat)/$1mooh/g;

print "$test";

Dat levert keurig "<blaat> mooh <b> mooh </b> </blaat>" op. Hij is niet helemaal perfect: als blaat voorkomt voor wat voor html tag dan ook, wordt ie niet gereplaced. Maar daar valt wel een tweede regex voor te verzinnen.

[Tom]

Sinds wanneer ben jij op normale tijden wakker meneer Ftl.F? zie je al een paar dagen s'morgens al reageren

[Femme]

Thanks Onno

[[ti]]

en dan nog 'n klein bugje met de hilighting-code: zoek bv es op username [ti] ( http://tweakers.net/nieuws/15586?&highlight=%5Bti%5D ) .. dan word de t en de i los van elkaar gehighlight en de brackets worden genegeerd.

[Femme]

[] () /\ doe ik nu ook escapen.

[Le Mol]

Als je '<b' of '<a' als highlight opgeeft wordt het ook best wel een zooitje

misschien ook '<' maar buiten de highlight laten ofzo...

[Verwijderd]

Gevonden

[topic=120380]

Tenks

[Burat]

Gevonden
Keyword highlighting BUG

Tenks

Que?

[Verwijderd]

sorry..verkeerde thread

[Marcel van Veen]

Op maandag 05 februari 2001 14:28 schreef Onno het volgende:

[..]

Dat levert keurig "<blaat> mooh mooh </blaat>" op. Hij is niet helemaal perfect: als blaat voorkomt voor wat voor html tag dan ook, wordt ie niet gereplaced. Maar daar valt wel een tweede regex voor te verzinnen.

Als we dan toch bezig zijn, iemand al een idee voor de tweede regexp?

[Onno]

Gewoon '>' voor de te replacen string plakken en die achteraf weer weghalen?

[Marcel van Veen]

Te simpel

thanx Onno

[Marcel van Veen]

Hmm, hoe maak ik dit zootje nou niet-greedy dan? Omzetten naar een preg_replace, maar dat gaat niet goed.....
* Marcel van Veen doet nog veel meer op regexp's dan Femme

[Onno]

Hmm...

http://www.tweakers.net/nieuws/16991?highlight=ibm

Niet elke IBM wordt gehighlight... foutje.

Tweede poging:

code:

$highlight = "ibm";
$bla =~ s|($highlight)(?![^<]*>)|<font color="red">$1</font>|gi;

Wat in php zoiets zou worden:

code:

$highlight = "ibm";
$bla = preg_replace("|($highlight)(?![^<]*>)|i","<font color=\"red\">$1</font>",$bla);

[Onno]

(als er nog een Femme rondloopt toevallig: ^^^)

[Femme]

Ja, maar die gaat nu een winterslaap houden .

Ik ben al sinds 2:30 bezig en daarvoor maar 4 uur geslapen.

[Onno]

Hmm... ok. Slaap ze.

* Onno wou alleen maar even weten of dit draadje niet toevallig in alle drukte aan Femmes aandacht ontsnapt was.

[Onno]

Al klaar met winterslaap?

[Floor-is]

Dat zal toch wel

[AntonT]

Op zaterdag 23 juni 2001 12:59 schreef Onno het volgende:
Al klaar met winterslaap?

Femme is nu online iig...

[Femme]

Done. thankx Onno