[React] Herparsing in list_discussions

[crisp]

In list_discussions is {discussion_firstmessage_content} beschikbaar, maar deze wordt niet geherparsed indien de versie niet up-to-date is.

[Spider.007]

Dat is consistent met het gedrag in list_messages; dus we moeten maar eens even kijken wanneer we dit consistent gaan aanpassen overal

[crisp]

Punt is dat we vanuit l33t pushmessages genereren en die unparsed in de tabel proppen zonder version, we vertrouwen daarmee dus op de herparsing feature.

[Spider.007]

Doe daarna even een list_discussion hit en dat probleem is opgelost toch? Het is toch ook niet zo dat dit verandert is oid?

[crisp]

Spider.007 schreef op maandag 28 augustus 2006 @ 10:19:
Doe daarna even een list_discussion hit en dat probleem is opgelost toch? Het is toch ook niet zo dat dit verandert is oid?

Het is in zoverre veranderd dat je vroegah alleen list_pushmessages had, en die herparsede wel (net als pm_discussion dat doet). list_discussions gebruikmakend van {discussion_firstmessage_content} doet dat niet, maar dat is wel de manier om de oude list_pushmessages-like view te genereren.

De vergelijking met list_messages gaat niet op, dan moet je het vergelijken met list_topics waarbij je {topic_firstmessage_content} ergens in gebruikt.

[ Voor 20% gewijzigd door crisp op 28-08-2006 10:24 ]

[chem]

Ik vind het geen bug; hoogstens een feature.

Als je buiten React om de tabellen manipuleert, dan kunnen we daar natuurlijk geen 'ondersteuning' op geven. Normaliter zou hoogstens de inhoud verouderd zijn, tov afwezig.

Je hebt wel gelijk dat die data nog steeds accuraat moet zijn, maar in 1.9.x zie ik ons dat niet nog overal inbouwen.

[crisp]

Het is ook geen 'big issue'
Sjon had het over een mogelijkheid om zelf acties in de admin te integreren; dat is natuurlijk ook een overweging, zeker als we op die manier eenvoudig direct de RML-parser kunnen aanspreken.
Ik denk dat het al veel scheelt als we al enkel een nl2br over de raw_content heenhalen bij het vullen van de parsed content.

Edit: dat laatste gedaan, dus wmb afdoende 'solved' atm

[ Voor 29% gewijzigd door crisp op 31-08-2006 11:02 ]