[Sugar-devel] Journal choking on invalid / missing metadata (was: Re: 11.3.0 release candidate 4 (build 883) released)

[Gonzalo Odiard]

I have already filled a ticket http://dev.laptop.org/ticket/11372
I have attached a file with the content of the Journal (only 4 entries) and
you can see
what the problem is. More info in the ticket.

Gonzalo

On Tue, Nov 1, 2011 at 9:54 AM, Sascha Silbe <
sascha-ml-reply-to-2011-4 at silbe.org> wrote:

> Excerpts from Gonzalo Odiard's message of 2011-11-01 13:09:55 +0100:
>
> [Trimmed the CC list to spare others from the gory details]
>
> > Reproduced here, datastore.log does not show anything, but shell.log
> show:
> [...]
>
> Thanks!
>
> Can you file a ticket at bugs.sl.o (if you use the link from my previous
> email most of the fields should get pre-filled), please?
>
> It would be useful to install IPython so we'll get Tracebacks annotated
> with the values of all the parameters, especially the value of object_id
> passed to
> jarabe.journal.journalactivity.JournalActivity.__detail_clicked_cb().
>
> Assuming it shows object_id='' as I suspect, please apply the following
> patch to provide further debugging output:
>
> diff --git i/src/jarabe/journal/model.py w/src/jarabe/journal/model.py
> index 5285a7c..1f45683 100644
> --- i/src/jarabe/journal/model.py
> +++ w/src/jarabe/journal/model.py
> @@ -36,6 +36,7 @@ import gconf
>  from sugar import dispatch
>  from sugar import mime
>  from sugar import util
> +from sugar.logger import trace
>
>
>  DS_DBUS_SERVICE = 'org.laptop.sugar.DataStore'
> @@ -220,6 +221,7 @@ class DatastoreResultSet(BaseResultSet):
>
>         BaseResultSet.__init__(self, query, page_size)
>
> +    @trace()
>     def find(self, query):
>         entries, total_count = _get_datastore().find(query, PROPERTIES,
>                                                      byte_arrays=True)
>
>
> You will need to set SUGAR_LOGGER_LEVEL to 'trace' (or 'all'), e.g. via
> ~/.sugar/debug. Please attach the shell.log will the new information to
> the ticket.
>
> A copy of the data store - or even just the single entry that triggers
> this behaviour - would still be useful.
>
> Sascha
>
> --
> http://sascha.silbe.org/
> http://www.infra-silbe.de/
>
> _______________________________________________
> Sugar-devel mailing list
> Sugar-devel at lists.sugarlabs.org
> http://lists.sugarlabs.org/listinfo/sugar-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20111101/ab990550/attachment.html>