← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/xml4.py

  • Committer: Andrew Bennetts
  • Date: 2007-03-26 06:24:01 UTC
  • mto: This revision was merged to the branch mainline in revision 2376.
  • Revision ID: andrew.bennetts@canonical.com-20070326062401-k3nbefzje5332jaf
Deal with review comments from Robert:

  * Add my name to the NEWS file
  * Move the test case to a new module in branch_implementations
  * Remove revision_history cruft from identitymap and test_identitymap
  * Improve some docstrings

Also, this fixes a bug where revision_history was not returning a copy of the
cached data, allowing the cache to be corrupted.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/xml4.py
17
17
from bzrlib.xml_serializer import ElementTree, SubElement, Element, Serializer
18
18
from bzrlib.inventory import ROOT_ID, Inventory, InventoryEntry
19
19
import bzrlib.inventory as inventory
20
 
from bzrlib.revision import Revision
 
20
from bzrlib.revision import Revision        
21
21
from bzrlib.errors import BzrError
22
22
 
23
23
 
24
24
class _Serializer_v4(Serializer):
25
25
    """Version 0.0.4 serializer
26
26
 
27
 
    You should use the serializer_v4 singleton.
28
 
    
29
 
    v4 serialisation is no longer supported, only deserialisation.
30
 
    """
 
27
    You should use the serializer_v4 singleton."""
31
28
    
32
29
    __slots__ = []
33
30
    
 
31
    def _pack_inventory(self, inv):
 
32
        """Convert to XML Element"""
 
33
        # v4 serialization is not used any more.
 
34
        raise NotImplementedError(self._pack_inventory)
 
35
        e = Element('inventory')
 
36
        e.text = '\n'
 
37
        if inv.root.file_id not in (None, ROOT_ID):
 
38
            e.set('file_id', inv.root.file_id)
 
39
        for path, ie in inv.iter_entries():
 
40
            e.append(self._pack_entry(ie))
 
41
        return e
 
42
 
 
43
 
34
44
    def _pack_entry(self, ie):
35
45
        """Convert InventoryEntry to XML element"""
36
46
        e = Element('entry')
50
60
        # for now, leaving them as null in the xml form.  in a future
51
61
        # version it will be implied by nested elements.
52
62
        if ie.parent_id != ROOT_ID:
 
63
            assert isinstance(ie.parent_id, basestring)
53
64
            e.set('parent_id', ie.parent_id)
54
65
 
55
66
        e.tail = '\n'
57
68
        return e
58
69
 
59
70
 
60
 
    def _unpack_inventory(self, elt, revision_id=None):
 
71
    def _unpack_inventory(self, elt):
61
72
        """Construct from XML Element
62
 
 
63
 
        :param revision_id: Ignored parameter used by xml5.
64
73
        """
 
74
        assert elt.tag == 'inventory'
65
75
        root_id = elt.get('file_id') or ROOT_ID
66
76
        inv = Inventory(root_id)
67
77
        for e in elt:
73
83
 
74
84
 
75
85
    def _unpack_entry(self, elt):
 
86
        assert elt.tag == 'entry'
 
87
 
76
88
        ## original format inventories don't have a parent_id for
77
89
        ## nodes in the root directory, but it's cleaner to use one
78
90
        ## internally.
129
141
            for i, parent_id in enumerate(rev.parents):
130
142
                p = SubElement(pelts, 'revision_ref')
131
143
                p.tail = '\n'
 
144
                assert parent_id
132
145
                p.set('revision_id', parent_id)
133
146
                if i < len(rev.parent_sha1s):
134
147
                    p.set('revision_sha1', rev.parent_sha1s[i])
156
169
 
157
170
        if pelts:
158
171
            for p in pelts:
 
172
                assert p.tag == 'revision_ref', \
 
173
                       "bad parent node tag %r" % p.tag
159
174
                rev.parent_ids.append(p.get('revision_id'))
160
175
                rev.parent_sha1s.append(p.get('revision_sha1'))
161
176
            if precursor:
162
177
                # must be consistent
163
178
                prec_parent = rev.parent_ids[0]
 
179
                assert prec_parent == precursor
164
180
        elif precursor:
165
181
            # revisions written prior to 0.0.5 have a single precursor
166
182
            # give as an attribute