Remove annotation storage in behavior fields #136

Closed
wants to merge 16 commits into
from

Projects

None yet

2 participants

@pysailor
Collaborator

Follow up from #95

For the trivial data fields (non-datetime related), expose the field directly on the object (no need to wrap them in the behavior first)

thet and others added some commits Feb 11, 2014
@thet thet start annotation-to-attribute storage migration step 158cc2a
@pysailor pysailor don't use annotation storage for Location, Attendees and Contact a15bcf5
@pysailor pysailor update test to check for behaviour-based marker interface 3aaef55
@pysailor pysailor BBB for the old marker interfaces for Location, Attendees and Contact 5f31909
@pysailor pysailor Revert "BBB for the old marker interfaces for Location, Attendees and…
… Contact"

This reverts commit 5f31909.
00f665b
@pysailor pysailor make sure to actually include the upgrades for dx 5de7596
@pysailor pysailor added upgrade step to migrate the annotation storage from behavior fi…
…elds
f92dbf5
@pysailor pysailor Added a test for the annotation to attribute storage migration d113930
@pysailor pysailor add a test that checks that already existing values don't get overwri…
…tten by the storage migration
4f30ab3
@pysailor pysailor Merge branch 'plone5' into remove-annotation-storage-95 5ac1cc9
@pysailor pysailor nicer descriptions in the tests 2bda86a
@pysailor pysailor Remove the 3 deprecated interfaces for the behaviors. They were only …
…used in tests
31dc8c4
@pysailor pysailor In the upgrade step, if we modified content, we should actually throw…
… the ObjectModified event
3deedf0
@pysailor pysailor we can simplify the indexer for SearchableText. We don't need to appl…
…y the IEventSummary behavior, since

the data (text) is already provided by the EventAccessor
aa7a21b
@pysailor pysailor also move the EventSummary (text) from Annotation to Attribute storage 41b9fc5
@pysailor pysailor Merge branch 'master' into remove-annotation-storage-95 cc5a354
@thet thet added a commit that closed this pull request Feb 12, 2014
@thet thet Use attribute storage instead of annotation storage in all Dexterity …
…behaviors. Closes #136, #95, Refs #20.
5c2f7af
@thet thet closed this in 5c2f7af Feb 12, 2014
@thet thet deleted the remove-annotation-storage-95 branch Feb 12, 2014
@jaroel jaroel added a commit that referenced this pull request Feb 12, 2014
@jaroel jaroel Merge branch 'master' into plip13476-mockup
* master: (41 commits)
  update test to expect correct default value (introduced in my last commit 483f91d)
  Provide tuple as default value for Tuple field
  Replace RecurrenceField with plain Text field in the dx recurrence behavior. This reverts the change from 1.0rc2. We don't use form schema hints but an adapter to configure the widget. Closes #137, Fixes #131.
  Use attribute storage instead of annotation storage in all Dexterity behaviors. Closes #136, #95, Refs #20.
  update Changes for #131
  Refs #131 actually we can simply replace the field that stores the recurrence. No upgrade step necessary, because existing data is kept
  also move the EventSummary (text) from Annotation to Attribute storage
  changelog note for #123
  we can simplify the indexer for SearchableText. We don't need to apply the IEventSummary behavior, since the data (text) is already provided by the EventAccessor
  In the upgrade step, if we modified content, we should actually throw the ObjectModified event
  Remove the 3 deprecated interfaces for the behaviors. They were only used in tests
  nicer descriptions in the tests
  add a test that checks that already existing values don't get overwritten by the storage migration
  Added a test for the annotation to attribute storage migration
  added upgrade step to migrate the annotation storage from behavior fields
  make sure to actually include the upgrades for dx
  more version fixes
  back to development
  prepare release
  prepare release
  ...

Conflicts:
	plone/app/event/dx/behaviors.py
dc88b3f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment