~lifeless/python-oops-tools/bug-879309

« back to all changes in this revision

Viewing changes to src/oopstools/oops/models.py

  • Committer: Robert Collins
  • Date: 2011-10-24 05:21:01 UTC
  • Revision ID: robertc@robertcollins.net-20111024052101-m1er1qzem8v9wibg
Do not use url for pageid when pageid is missing.

Show diffs side-by-side

added added

removed removed

Lines of Context:
379
379
        oopsid = oopsid,
380
380
        prefix = prefix,
381
381
        date = oops_date.replace(microsecond=0),
382
 
        # Missing pageids are urls because that suits our queries.
383
 
        pageid = oops.get('topic') or url,
 
382
        # Missing topics are '': will group all such reports together in some
 
383
        # ways, but thats tolerable vs something (near) unique like url: that
 
384
        # prevents aggregation.
 
385
        pageid = oops.get('topic') or '',
384
386
        url = url,
385
387
        duration = duration,
386
388
        informational = informational,