Creating a new Journal entry can have the wrong order

[erikos]

See ​http://dev.laptop.org/ticket/10905 which deals with the case where we are hitting the issue when "Copying an item from external device to Journal" and #4305 where we have seen this issue when we create a screenshot in the Home View.

[erikos]

Please see ​http://dev.laptop.org/ticket/10905 for a first diagnosis of this issue.

[erikos]

This patch does force to flush the index, otherwise we do have the 5 seconds timeout

[erikos]

Test:

• on the Home View, take a screen-shot

• switch directly to the Journal

---> the new item is at the bottom

• on the Home View, take a screen-shot, wait there a few seconds

• switch to the Journal

---> the new item is at the top, correctly ordered

The patch above does make both cases work correctly.

[manuq]

Replying to erikos:

Test:

• on the Home View, take a screen-shot

• switch directly to the Journal

---> the new item is at the bottom

• on the Home View, take a screen-shot, wait there a few seconds

• switch to the Journal

---> the new item is at the top, correctly ordered

The patch above does make both cases work correctly.

Great Simon, the testcase passes with the patch applied for me, in sugar-build and in the XO.

[manuq]

I confirm #4305, entries made by doing screenshots, passes too.

Today I have seen this bug in 12.1.0, so it is with us at least since then.

[erikos]

Pushed as datastore: 56171a8e46d53d41ee5f056a9155975917ebc379 after some more testing.

[godiard]

Another alternative solution

[godiard]

Attached a different solution proposal.

After check the sequence of events, I found than at times,
the Created dbus signal before the index was flushed.

This patch check if the index is updated, and if not, flush it
immediatly before the signal is emited.

[dnarvaez]

Milestone 0.98 deleted