Repository navigation
Update cosmo.properties - #1
Merged
Merged
Conversation
wsdwl
added a commit
that referenced
this pull request
Jun 18, 2015
Update cosmo.properties - has been overlocked. Thanks!
f1-outsourcing
pushed a commit
to f1-outsourcing/cosmo
that referenced
this pull request
Aug 29, 2026
fix: emit RFC 6578 change-log entries (D+C) on cross-collection MOVE
StandardContentService#moveItem now writes a DAV:sync-collection
tombstone (D-row) at the source collection and a live member (C-row)
at the destination, per RFC 6578 §3.2/§5 (a rename/move is reported
as delete at the old href + create at the new href).
- D-row logged before removeItemFromCollection (source tombstone)
- C-row logged after addItemToCollection (destination live member)
- Guarded by crossCollection so same-parent renames (single M-row via
updateContent) do not double-log
- Applied in both the locking ContentItem branch and the plain branch
Test (I2):
- Rewrote crossCollectionMoveEmitsTombstoneInSourceAndLiveMemberInTarget
to lock the RFC-compliant shape: exactly one bare-404 tombstone in
A's incremental sync, exactly one live 200 getetag entry in B's
- Fixed three test-bug helpers:
* OutputKeys is a top-level class (not Transformer.OutputKeys)
* hrefStartsWithPrefix now decodes the percent-encoded collectionUri
* B-side 200 lives INSIDE the propstat (Cosmo/RFC 4918 shape), not
as a response-level DAV:status
Docs: rfc-compliance.md (Known Limitations mam-dev#1, Roadmap mam-dev#5) and
synccollection-testcases.md (I2 rows, notes, status, traceability)
updated to reflect the implemented behavior.
Full cosmo-core suite: 395/395 green.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
please use example.com for domain names in configuration files.