Fatal error: Cannot use object of type error as array

[Sando]

Eerder was dit geen probleem, nu krijg ik een fout.

[edit]
tieptiep
[verstuur]
browser history back
tieptiep
[verstuur]

Fatal error: Cannot use object of type error as array in /mnt/web/react/got/react/global/non-www/classes/message.class.inc.php on line 259

Dan weet je ervan.

[crisp]

de veroorzaker: browser history back

hierdoor wordt dus het versienummer in het formulier niet bijgewerkt en gaat er eea mis bij de verwerking. Dit versienummer wordt gebruikt voor multi-user editting om changes te kunnen mergen. Eens kijken hoe ik dat kan oplossen (naast het netjes opvangen van die error )

[ACM]

Wat nu ook niet heel netjes is, is dat edits verloren kunnen gaan.

Stel gebruiker A opent een bericht.
Gebruiker B opent hetzelfde bericht.

Gebruiker B typt wat en stuurt dit op.
Gebruiker A typt wat en stuurt dit op.

De wijzigingen van gebruiker A zijn nu niet doorgevoerd, dit kan je eenvoudig testen door een bericht in twee browservensters/tabs tegelijk te openen.

[Sando]

Multi-user editing? Dat is zeker iets wat niet voor de normale user relevant is?
(nieuwschierig)

[crisp]

ACM schreef op zondag 14 maart 2010 @ 10:37:
Wat nu ook niet heel netjes is, is dat edits verloren kunnen gaan.

Stel gebruiker A opent een bericht.
Gebruiker B opent hetzelfde bericht.

Gebruiker B typt wat en stuurt dit op.
Gebruiker A typt wat en stuurt dit op.

De wijzigingen van gebruiker A zijn nu niet doorgevoerd, dit kan je eenvoudig testen door een bericht in twee browservensters/tabs tegelijk te openen.

Dat is alleen het geval als het mergen van de twee diffs een conflict oplevert, da's een known issue.

Sando schreef op zondag 14 maart 2010 @ 11:00:
Multi-user editing? Dat is zeker iets wat niet voor de normale user relevant is?
(nieuwschierig)

Wel hoor, je kan een startpost van een topic met meerdere users onderhouden.

[ Voor 20% gewijzigd door crisp op 14-03-2010 11:29 ]

[ACM]

crisp schreef op zondag 14 maart 2010 @ 11:27:
Dat is alleen het geval als het mergen van de twee diffs een conflict oplevert, da's een known issue.

Ok, dat scheelt, alleen zou het wel handig zijn als de 2e daar dan een melding van krijgt

[crisp]

ACM schreef op zondag 14 maart 2010 @ 11:44:
[...]

Ok, dat scheelt, alleen zou het wel handig zijn als de 2e daar dan een melding van krijgt

Ja, daar staat al een FIXME in de code

In ieder geval denk ik dat het probleem van de topicstarter het beste is op te lossen door geen diff-merging te doen als je zelf ook al de laatste updater was.

[crisp]

Fixed in SVN