Afbeeldingen quoten

[Ger]

Ik zie in de output dit staan:

Code: Selecteer alles

<!-- s:D --><img src=\"{SMILIES_PATH}/icon_e_biggrin.gif\" alt=\":D\" title=\"Zeer gelukkig\" /><!-- s:D -->

Daar zitten een paar dingen fout in:

1. De comment s:D (volgens mij hoort die comment daar niet eens, en met :D bestaat niet). Staat me bij dat die comments in de message parser of in bbcode.php ergens gedefinieerd worden, maar ik heb nu even geen gelegenheid dat na te gaan (verwacht dat ook niet voor woensdag te kunnen doen).
2. {SMILIES_PATH} moet zijn ./images/smilies
3. De aanhalingstekens worden geëscaped

Ik gok haast dat het patroon van de reguliere expressie verkeerd gaat. Hoewel we "ungreedy" aangeven (wat dus inhoudt dat hij tussen de afzonderlijke quotes moet kijken en niet tussen de eerste quotestart en laatste quoteclose), lijkt hij toch greedy te zijn.

Veel verder kan ik er nu even niet op ingaan, op zijn vroegst woensdag. Maar misschien is dit al genoeg info voor iemand om verder mee te komen.

[mosymuis]

Het lukt me niet om dit te reproduceren.

//edit
Dat is nog eens toevallig samen posten^^

[mosymuis]

Ik zie in de output dit staan:

Code: Selecteer alles

<!-- s:D --><img src=\"{SMILIES_PATH}/icon_e_biggrin.gif\" alt=\":D\" title=\"Zeer gelukkig\" /><!-- s:D -->

Daar zitten een paar dingen fout in:

1. De comment s:D (volgens mij hoort die comment daar niet eens, en met :D bestaat niet). Staat me bij dat die comments in de message parser of in bbcode.php ergens gedefinieerd worden, maar ik heb nu even geen gelegenheid dat na te gaan (verwacht dat ook niet voor woensdag te kunnen doen).
2. {SMILIES_PATH} moet zijn ./images/smilies
3. De aanhalingstekens worden geëscaped

Ik gok haast dat het patroon van de reguliere expressie verkeerd gaat.

Dit is normale opslag van een quote in een stuk tekst in phpbb_posts, op de quotes na. Die horen daar niet. Het hoort te zijn:

Code: Selecteer alles

<!-- s:D --><img src="{SMILIES_PATH}/icon_e_biggrin.gif" alt=":D" title="Zeer gelukkig" /><!-- s:D -->

Je moet dus uitzoeken waarom de quotes worden gezet.

[Tazmanian]

Op phpbb.com is er momenteel iemand bezig aan een Mod die doet wat je vraagt.
Ziet momenteel in RC, zal waarschijnlijk binnenkort in de Mod database verschijnen.
http://www.phpbb.com/community/viewtopi ... &t=1231365

[mosymuis]

Op phpbb.com is er momenteel iemand bezig aan een Mod die doet wat je vraagt.
Ziet momenteel in RC, zal waarschijnlijk binnenkort in de Mod database verschijnen.
http://www.phpbb.com/community/viewtopi ... &t=1231365

Um, dat is dus het werk dat we hier op pagina 1 besproken hebben. Ger heeft zich blijkaar opgeworpen als MOD auteur daar.

[Bee]

Ik weet zelf overigens niet zo goed waar ik het in de reguliere expressies moet zoeken, ik ben daar eh, heel slecht in

[Ger]

Dit is normale opslag van een quote in een stuk tekst in phpbb_posts, op de quotes na. Die horen daar niet. Het hoort te zijn:

Code: Selecteer alles

<!-- s:D --><img src="{SMILIES_PATH}/icon_e_biggrin.gif" alt=":D" title="Zeer gelukkig" /><!-- s:D -->

Je moet dus uitzoeken waarom de quotes worden gezet.

Nee, niet helemaal, want {SMILIES_PATH}/ moet natuurlijk omgezet worden naar een ./. De smilie wordt dus gewoon niet geparsed zoals het zou moeten.
Wat je zou kunnen proberen om dat op te lossen is de code van de mod iets omhoog schuiven. Het staat nu achter deze regel:

Code: Selecteer alles

		$in = str_replace("\r\n", "\n", str_replace('\"', '"', trim($in)));

En als je de code voor deze regel zet zou het kunnen helpen vwb de \". Blijf je waarschijnlijk nog wel zitten met die {SMILIES_PATH}/. Ik kan het echter niet testen zelf, want ik krijg het ook niet gereproduceerd op mijn forum. Overigens gaat het wel mis als ik tussen 2 quotes een afbeelding invoeg met img-tags, dus iets is er nog wel mis met het pattern. Daar ga ik echter nu even niet meer induiken.

Um, dat is dus het werk dat we hier op pagina 1 besproken hebben. Ger heeft zich blijkaar opgeworpen als MOD auteur daar.

Klopt, het stond me bij dat ik die vraag al eerder was tegengekomen, dus leek het me net zo handig om dat even in een mod te gieten voor het grote publiek. Mosymuis staat trouwens ook bij de author notes genoemd.

[mosymuis]

Nee, niet helemaal, want {SMILIES_PATH}/ moet natuurlijk omgezet worden naar een ./.

Dat gebeurt bij de weergave van een bericht; zo ^^ staat het in de database opgeslagen, zoals het hoort. Dat is niet je probleem; de quotes zijn je probleem. De vraag is waarom die erbij worden gezet.

Ik kan het echter niet testen zelf, want ik krijg het ook niet gereproduceerd op mijn forum.

Ik ook niet; dus dan is het voor mij zoeken in het donker. Ik kan er zo dus geen zinnig woord over zeggen.

Mosymuis staat trouwens ook bij de author notes genoemd.

Mij noemen als "inspiration" is niet helemaal hetzelfde als schrijver, denk je wel? Ik zou het eerder omdraaien aangezien jij mij van wat ideëen voorzag.

[Coen]

Kunnen jullie het proberen te reproduceren middels dit bericht;

Code: Selecteer alles

[quote][ [url=http://www.google.nl/intl/nl_nl/images/logo.gif]afbeelding[/url] ][/quote]
Images binnen quotes zijn linkjes! :D

[quote="Coen"]Voor alle testjes die je uit wilt voeren met bijvoorbeeld een BBcode.

Nog meer tekst.[/quote]

[img]http://www.google.nl/intl/nl_nl/images/logo.gif[/img]

:)

En dan wel met de laatste wijziging op de vorige pagina doorgevoerd natuurlijk!

[mosymuis]

Kunnen jullie het proberen te reproduceren middels dit bericht;

nee, dat gaat gewoon goed, ik zie geen corrupte smilies verschijnen na het paatsen en quoten

[Ger]

Dat met die smilies krijg ik met de beste wil van de wereld niet gereproduceerd... Misschien heeft het iets te maken met de PHP versie ofzo, geen idee.

Overigens gaat het wel mis als ik tussen 2 quotes een afbeelding invoeg met img-tags, dus iets is er nog wel mis met het pattern.

Dat was dus als je een bericht hebt als

Code: Selecteer alles

[quote]sdojfsdf [/quote]
[img]www.example.com/picture1.gif[/img]
[quote]sdojfsdf [/quote]

Dat heb ik als volgt opgelost. Als eerste verwijder je de code die je in eerste instantie hebt toegevoegd als mod. Vervolgens zoek je in includes/message_parser.php naar:

Code: Selecteer alles

(Lijn 36-40)

	/**
	* Parse BBCode
	*/
	function parse_bbcode()
	{

En zet daar onder:

Code: Selecteer alles

		//Start MOD Convert quoted images to url 	
		preg_match_all('#\[quote(.*?)\](.*?)\[/quote\]#si', $this->message, $qmatch );
		foreach ($qmatch[2]as $qmatchin){
			$qres = preg_replace('#\[url=(.*(jpg|jpeg|gif|png|bmp))\]\[img\].*\[/img\]\[/url\]#iU', '[ [url=\1]afbeelding[/url] ]', $qmatchin);
			$qres = preg_replace('#\[url=.*\]\[img\](.*)\[/img\]\[/url\]#iU', '[ [url=\1]afbeelding[/url] ]', $qmatchin);
			$qres = preg_replace('#\[img\](.*)\[/img\]#iU', '[ [url=\1]afbeelding[/url] ]', $qmatchin);
			$this->message = str_replace($qmatchin, $qres, $this->message);
		}
		//End MOD Convert quoted images to url

Lost dat misschien ook het probleem van de smilies op? Indien nee, is mijn eerdere suggestie al geprobeerd? (en wat was het resultaat dan?)

[Coen]

Volgens mij werkt het helemaal! Ger, je bent geweldig!

Ik moet toch eens zelf wat meer in dat regexp gaan doen, kon toch wel eens handig zijn!

Het resultaat is nu te zien in mijn testtopic!

[Ger]

FIjn zo!