Merge lp:~cjwatson/storm/reference-set-update-zcml into lp:storm

Proposed by Colin Watson
Status: Merged
Merged at revision: 549
Proposed branch: lp:~cjwatson/storm/reference-set-update-zcml
Merge into: lp:storm
Diff against target: 28 lines (+16/-2)
1 file modified
storm/zope/configure.zcml (+16/-2)
To merge this branch: bzr merge lp:~cjwatson/storm/reference-set-update-zcml
Reviewer Review Type Date Requested Status
Tom Wardill (community) Approve
Storm Developers Pending
Review via email: mp+380839@code.launchpad.net

Commit message

Update ZCML for BoundReferenceSet to include __getitem__ and is_empty.

Description of the change

Also reorder the attributes listed there to be in the same order as the implementation in BoundReferenceSetBase.

To post a comment you must log in.
Revision history for this message
Tom Wardill (twom) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'storm/zope/configure.zcml'
2--- storm/zope/configure.zcml 2009-03-05 11:17:47 +0000
3+++ storm/zope/configure.zcml 2020-03-18 17:07:10 +0000
4@@ -24,8 +24,22 @@
5 </class>
6
7 <class class="storm.references.BoundReferenceSet">
8- <allow attributes="__iter__ find first last one any values order_by
9- count __contains__" />
10+ <!-- Keep these in the order found in the implementation of
11+ storm.references.BoundReferenceSetBase. -->
12+ <allow attributes="
13+ find
14+ __iter__
15+ __getitem__
16+ __contains__
17+ is_empty
18+ first
19+ last
20+ any
21+ one
22+ values
23+ order_by
24+ count
25+ " />
26 </class>
27
28 <class class="storm.references.BoundIndirectReferenceSet">

Subscribers

People subscribed via source and target branches

to status/vote changes: