Launchpad itself

Implement new cachedproperty API

Bug #623421 reported by Gavin Panella
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Gavin Panella
Launchpad itself 10.09 "Hungry Hippo"

Bug Description

The current cachedproperty API is a bit confusing and hairy.

Related branches

lp:~allenap/launchpad/cache-experiment
Tim Penhey (community): Approve
Henning Eggers (community): Approve (code)
Diff: 922 lines (+455/-104)
12 files modified
lib/canonical/configure.zcml (+1/-10)
lib/canonical/database/sqlbase.py (+50/-37)
lib/lp/registry/doc/personlocation.txt (+4/-3)
lib/lp/registry/doc/teammembership.txt (+2/-2)
lib/lp/registry/model/distribution.py (+7/-12)
lib/lp/registry/model/person.py (+32/-31)
lib/lp/registry/tests/test_distribution.py (+7/-7)
lib/lp/services/configure.zcml (+19/-0)
lib/lp/services/doc/propertycache.txt (+153/-0)
lib/lp/services/propertycache.py (+174/-0)
lib/lp/services/tests/test_doc.py (+4/-0)
lib/lp/soyuz/model/archive.py (+2/-2)
lp:~allenap/launchpad/cache-experiment-roll-out
Graham Binns (community): Approve (code)
Robert Collins (community): Approve
Diff: 2880 lines (+248/-457)
114 files modified
.ctags (+3/-0)
lib/canonical/cachedproperty.py (+0/-229)
lib/canonical/database/sqlbase.py (+0/-6)
lib/canonical/launchpad/browser/launchpad.py (+1/-1)
lib/canonical/launchpad/utilities/geoip.py (+1/-1)
lib/canonical/launchpad/webapp/error.py (+1/-1)
lib/canonical/launchpad/webapp/login.py (+1/-1)
lib/canonical/launchpad/webapp/servers.py (+1/-1)
lib/canonical/lazr/feed/feed.py (+17/-8)
lib/canonical/tests/test_cachedproperty.py (+0/-15)
lib/canonical/widgets/popup.py (+8/-7)
lib/lp/answers/browser/faqcollection.py (+1/-1)
lib/lp/answers/browser/question.py (+1/-1)
lib/lp/answers/browser/questiontarget.py (+3/-3)
lib/lp/answers/notification.py (+1/-1)
lib/lp/app/browser/root.py (+1/-1)
lib/lp/archivepublisher/diskpool.py (+1/-1)
lib/lp/blueprints/browser/specification.py (+1/-1)
lib/lp/blueprints/browser/specificationgoal.py (+1/-1)
lib/lp/blueprints/browser/specificationtarget.py (+1/-1)
lib/lp/blueprints/browser/sprint.py (+1/-1)
lib/lp/bugs/browser/bug.py (+1/-1)
lib/lp/bugs/browser/bugalsoaffects.py (+1/-1)
lib/lp/bugs/browser/bugbranch.py (+1/-1)
lib/lp/bugs/browser/bugtarget.py (+1/-1)
lib/lp/bugs/browser/bugtask.py (+1/-1)
lib/lp/bugs/browser/bugtracker.py (+1/-1)
lib/lp/bugs/browser/cvereport.py (+1/-1)
lib/lp/bugs/browser/distribution_upstream_bug_report.py (+1/-1)
lib/lp/bugs/doc/externalbugtracker-mantis-csv.txt (+3/-1)
lib/lp/bugs/externalbugtracker/mantis.py (+1/-1)
lib/lp/bugs/externalbugtracker/rt.py (+1/-1)
lib/lp/bugs/model/bug.py (+13/-12)
lib/lp/bugs/model/bugattachment.py (+3/-3)
lib/lp/bugs/model/bugtask.py (+2/-2)
lib/lp/bugs/tests/externalbugtracker.py (+0/-8)
lib/lp/buildmaster/model/builder.py (+1/-1)
lib/lp/code/browser/bazaar.py (+1/-1)
lib/lp/code/browser/branch.py (+1/-1)
lib/lp/code/browser/branchlisting.py (+1/-1)
lib/lp/code/browser/branchmergeproposal.py (+1/-1)
lib/lp/code/browser/branchmergeproposallisting.py (+1/-1)
lib/lp/code/browser/branchvisibilitypolicy.py (+1/-1)
lib/lp/code/browser/codeimport.py (+1/-1)
lib/lp/code/browser/codeimportmachine.py (+1/-1)
lib/lp/code/browser/codereviewcomment.py (+1/-1)
lib/lp/code/browser/decorations.py (+1/-1)
lib/lp/code/browser/summary.py (+1/-1)
lib/lp/code/feed/branch.py (+1/-1)
lib/lp/code/model/branchvisibilitypolicy.py (+1/-1)
lib/lp/codehosting/codeimport/worker.py (+1/-1)
lib/lp/codehosting/puller/scheduler.py (+1/-1)
lib/lp/hardwaredb/scripts/hwdbsubmissions.py (+1/-1)
lib/lp/registry/browser/__init__.py (+1/-1)
lib/lp/registry/browser/announcement.py (+1/-1)
lib/lp/registry/browser/distribution.py (+2/-4)
lib/lp/registry/browser/distributionmirror.py (+1/-1)
lib/lp/registry/browser/distributionsourcepackage.py (+4/-3)
lib/lp/registry/browser/distroseries.py (+1/-1)
lib/lp/registry/browser/karma.py (+1/-1)
lib/lp/registry/browser/mailinglists.py (+1/-1)
lib/lp/registry/browser/milestone.py (+1/-1)
lib/lp/registry/browser/person.py (+6/-6)
lib/lp/registry/browser/pillar.py (+1/-1)
lib/lp/registry/browser/product.py (+3/-3)
lib/lp/registry/browser/productseries.py (+1/-1)
lib/lp/registry/browser/project.py (+1/-1)
lib/lp/registry/browser/structuralsubscription.py (+1/-1)
lib/lp/registry/browser/team.py (+1/-1)
lib/lp/registry/model/distribution.py (+5/-8)
lib/lp/registry/model/distroseries.py (+3/-5)
lib/lp/registry/model/mailinglist.py (+1/-1)
lib/lp/registry/model/person.py (+4/-22)
lib/lp/registry/model/product.py (+6/-3)
lib/lp/registry/vocabularies.py (+1/-1)
lib/lp/services/mailman/doc/staging.txt (+2/-1)
lib/lp/services/openid/browser/openiddiscovery.py (+1/-1)
lib/lp/services/propertycache.py (+41/-0)
lib/lp/services/salesforce/proxy.py (+1/-1)
lib/lp/services/tests/test_doc.py (+4/-2)
lib/lp/shipit.py (+4/-0)
lib/lp/soyuz/browser/archive.py (+2/-4)
lib/lp/soyuz/browser/archivesubscription.py (+1/-1)
lib/lp/soyuz/browser/build.py (+2/-2)
lib/lp/soyuz/browser/builder.py (+1/-1)
lib/lp/soyuz/browser/distributionsourcepackagerelease.py (+2/-2)
lib/lp/soyuz/browser/packagesearch.py (+1/-1)
lib/lp/soyuz/browser/publishing.py (+1/-1)
lib/lp/soyuz/model/archive.py (+1/-3)
lib/lp/soyuz/model/distroarchseriesbinarypackage.py (+2/-2)
lib/lp/soyuz/model/distroseriesbinarypackage.py (+6/-3)
lib/lp/soyuz/model/queue.py (+1/-1)
lib/lp/soyuz/model/sourcepackagerelease.py (+2/-4)
lib/lp/soyuz/scripts/ppareport.py (+1/-1)
lib/lp/soyuz/scripts/queue.py (+1/-1)
lib/lp/testing/__init__.py (+7/-2)
lib/lp/testopenid/browser/server.py (+7/-3)
lib/lp/translations/browser/distribution.py (+1/-1)
lib/lp/translations/browser/distroseries.py (+1/-1)
lib/lp/translations/browser/hastranslationimports.py (+1/-1)
lib/lp/translations/browser/language.py (+1/-1)
lib/lp/translations/browser/person.py (+1/-1)
lib/lp/translations/browser/poexportrequest.py (+1/-1)
lib/lp/translations/browser/pofile.py (+1/-1)
lib/lp/translations/browser/product.py (+1/-1)
lib/lp/translations/browser/productseries.py (+1/-1)
lib/lp/translations/browser/serieslanguage.py (+1/-1)
lib/lp/translations/browser/translationmessage.py (+1/-1)
lib/lp/translations/browser/translations.py (+1/-1)
lib/lp/translations/model/pofile.py (+1/-1)
lib/lp/translations/model/potemplate.py (+1/-1)
lib/lp/translations/model/translationmessage.py (+1/-1)
lib/lp/translations/utilities/translation_import.py (+1/-1)
utilities/sourcedeps.conf (+1/-1)
Henning Eggers (henninge)
affects: launchpad → launchpad-foundations
Revision history for this message
Launchpad QA Bot (lpqabot) wrote : Bug fixed by a commit

Fixed in stable r11495 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/11495>.

Changed in launchpad-foundations:
milestone: none → 10.09
tags: added: qa-needstesting
Changed in launchpad-foundations:
status: In Progress → Fix Committed
Robert Collins (lifeless)
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in launchpad-foundations:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.