/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/xml8.py

  • Committer: Vincent Ladeuil
  • Date: 2009-03-19 17:51:18 UTC
  • mto: This revision was merged to the branch mainline in revision 4169.
  • Revision ID: v.ladeuil+lp@free.fr-20090319175118-zdi3fwmv3wzuktfu
Fix UnavailableFTPTestServer for sites without medusa and pyftpdlib.

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2005, 2006, 2007, 2008 Canonical Ltd
 
2
#
1
3
# This program is free software; you can redistribute it and/or modify
2
4
# it under the terms of the GNU General Public License as published by
3
5
# the Free Software Foundation; either version 2 of the License, or
4
6
# (at your option) any later version.
5
 
 
 
7
#
6
8
# This program is distributed in the hope that it will be useful,
7
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
8
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
9
11
# GNU General Public License for more details.
10
 
 
 
12
#
11
13
# You should have received a copy of the GNU General Public License
12
14
# along with this program; if not, write to the Free Software
13
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
14
16
 
 
17
import cStringIO
 
18
import re
15
19
 
16
 
from bzrlib.xml_serializer import ElementTree, SubElement, Element, Serializer
 
20
from bzrlib import (
 
21
    cache_utf8,
 
22
    errors,
 
23
    inventory,
 
24
    revision as _mod_revision,
 
25
    trace,
 
26
    )
 
27
from bzrlib.xml_serializer import SubElement, Element, Serializer
17
28
from bzrlib.inventory import ROOT_ID, Inventory, InventoryEntry
18
 
import bzrlib.inventory as inventory
19
 
from bzrlib.revision import Revision        
 
29
from bzrlib.revision import Revision
20
30
from bzrlib.errors import BzrError
21
31
 
22
32
 
23
 
class Serializer_v5(Serializer):
24
 
    """Version 5 serializer
25
 
 
26
 
    Packs objects into XML and vice versa.
27
 
    """
28
 
    
 
33
_utf8_re = None
 
34
_unicode_re = None
 
35
_xml_escape_map = {
 
36
    "&":'&',
 
37
    "'":"'", # FIXME: overkill
 
38
    "\"":""",
 
39
    "<":"&lt;",
 
40
    ">":"&gt;",
 
41
    }
 
42
 
 
43
 
 
44
def _ensure_utf8_re():
 
45
    """Make sure the _utf8_re and _unicode_re regexes have been compiled."""
 
46
    global _utf8_re, _unicode_re
 
47
    if _utf8_re is None:
 
48
        _utf8_re = re.compile('[&<>\'\"]|[\x80-\xff]+')
 
49
    if _unicode_re is None:
 
50
        _unicode_re = re.compile(u'[&<>\'\"\u0080-\uffff]')
 
51
 
 
52
 
 
53
def _unicode_escape_replace(match, _map=_xml_escape_map):
 
54
    """Replace a string of non-ascii, non XML safe characters with their escape
 
55
 
 
56
    This will escape both Standard XML escapes, like <>"', etc.
 
57
    As well as escaping non ascii characters, because ElementTree did.
 
58
    This helps us remain compatible to older versions of bzr. We may change
 
59
    our policy in the future, though.
 
60
    """
 
61
    # jam 20060816 Benchmarks show that try/KeyError is faster if you
 
62
    # expect the entity to rarely miss. There is about a 10% difference
 
63
    # in overall time. But if you miss frequently, then if None is much
 
64
    # faster. For our use case, we *rarely* have a revision id, file id
 
65
    # or path name that is unicode. So use try/KeyError.
 
66
    try:
 
67
        return _map[match.group()]
 
68
    except KeyError:
 
69
        return "&#%d;" % ord(match.group())
 
70
 
 
71
 
 
72
def _utf8_escape_replace(match, _map=_xml_escape_map):
 
73
    """Escape utf8 characters into XML safe ones.
 
74
 
 
75
    This uses 2 tricks. It is either escaping "standard" characters, like "&<>,
 
76
    or it is handling characters with the high-bit set. For ascii characters,
 
77
    we just lookup the replacement in the dictionary. For everything else, we
 
78
    decode back into Unicode, and then use the XML escape code.
 
79
    """
 
80
    try:
 
81
        return _map[match.group()]
 
82
    except KeyError:
 
83
        return ''.join('&#%d;' % ord(uni_chr)
 
84
                       for uni_chr in match.group().decode('utf8'))
 
85
 
 
86
 
 
87
_to_escaped_map = {}
 
88
 
 
89
def _encode_and_escape(unicode_or_utf8_str, _map=_to_escaped_map):
 
90
    """Encode the string into utf8, and escape invalid XML characters"""
 
91
    # We frequently get entities we have not seen before, so it is better
 
92
    # to check if None, rather than try/KeyError
 
93
    text = _map.get(unicode_or_utf8_str)
 
94
    if text is None:
 
95
        if unicode_or_utf8_str.__class__ is unicode:
 
96
            # The alternative policy is to do a regular UTF8 encoding
 
97
            # and then escape only XML meta characters.
 
98
            # Performance is equivalent once you use cache_utf8. *However*
 
99
            # this makes the serialized texts incompatible with old versions
 
100
            # of bzr. So no net gain. (Perhaps the read code would handle utf8
 
101
            # better than entity escapes, but cElementTree seems to do just fine
 
102
            # either way)
 
103
            text = str(_unicode_re.sub(_unicode_escape_replace,
 
104
                                       unicode_or_utf8_str)) + '"'
 
105
        else:
 
106
            # Plain strings are considered to already be in utf-8 so we do a
 
107
            # slightly different method for escaping.
 
108
            text = _utf8_re.sub(_utf8_escape_replace,
 
109
                                unicode_or_utf8_str) + '"'
 
110
        _map[unicode_or_utf8_str] = text
 
111
    return text
 
112
 
 
113
 
 
114
def _get_utf8_or_ascii(a_str,
 
115
                       _encode_utf8=cache_utf8.encode,
 
116
                       _get_cached_ascii=cache_utf8.get_cached_ascii):
 
117
    """Return a cached version of the string.
 
118
 
 
119
    cElementTree will return a plain string if the XML is plain ascii. It only
 
120
    returns Unicode when it needs to. We want to work in utf-8 strings. So if
 
121
    cElementTree returns a plain string, we can just return the cached version.
 
122
    If it is Unicode, then we need to encode it.
 
123
 
 
124
    :param a_str: An 8-bit string or Unicode as returned by
 
125
                  cElementTree.Element.get()
 
126
    :return: A utf-8 encoded 8-bit string.
 
127
    """
 
128
    # This is fairly optimized because we know what cElementTree does, this is
 
129
    # not meant as a generic function for all cases. Because it is possible for
 
130
    # an 8-bit string to not be ascii or valid utf8.
 
131
    if a_str.__class__ is unicode:
 
132
        return _encode_utf8(a_str)
 
133
    else:
 
134
        return intern(a_str)
 
135
 
 
136
 
 
137
def _clear_cache():
 
138
    """Clean out the unicode => escaped map"""
 
139
    _to_escaped_map.clear()
 
140
 
 
141
 
 
142
class Serializer_v8(Serializer):
 
143
    """This serialiser adds rich roots.
 
144
 
 
145
    Its revision format number matches its inventory number.
 
146
    """
 
147
 
29
148
    __slots__ = []
30
 
    
31
 
    def _pack_inventory(self, inv):
32
 
        """Convert to XML Element"""
33
 
        e = Element('inventory',
34
 
                    format='5')
35
 
        e.text = '\n'
36
 
        if inv.root.file_id not in (None, ROOT_ID):
37
 
            e.set('file_id', inv.root.file_id)
 
149
 
 
150
    root_id = None
 
151
    support_altered_by_hack = True
 
152
    # This format supports the altered-by hack that reads file ids directly out
 
153
    # of the versionedfile, without doing XML parsing.
 
154
 
 
155
    supported_kinds = set(['file', 'directory', 'symlink'])
 
156
    format_num = '8'
 
157
    revision_format_num = None
 
158
 
 
159
    def _check_revisions(self, inv):
 
160
        """Extension point for subclasses to check during serialisation.
 
161
 
 
162
        :param inv: An inventory about to be serialised, to be checked.
 
163
        :raises: AssertionError if an error has occured.
 
164
        """
 
165
        if inv.revision_id is None:
 
166
            raise AssertionError()
 
167
        if inv.root.revision is None:
 
168
            raise AssertionError()
 
169
 
 
170
    def _check_cache_size(self, inv_size, entry_cache):
 
171
        """Check that the entry_cache is large enough.
 
172
 
 
173
        We want the cache to be ~2x the size of an inventory. The reason is
 
174
        because we use a FIFO cache, and how Inventory records are likely to
 
175
        change. In general, you have a small number of records which change
 
176
        often, and a lot of records which do not change at all. So when the
 
177
        cache gets full, you actually flush out a lot of the records you are
 
178
        interested in, which means you need to recreate all of those records.
 
179
        An LRU Cache would be better, but the overhead negates the cache
 
180
        coherency benefit.
 
181
 
 
182
        One way to look at it, only the size of the cache > len(inv) is your
 
183
        'working' set. And in general, it shouldn't be a problem to hold 2
 
184
        inventories in memory anyway.
 
185
 
 
186
        :param inv_size: The number of entries in an inventory.
 
187
        """
 
188
        if entry_cache is None:
 
189
            return
 
190
        # 1.5 times might also be reasonable.
 
191
        recommended_min_cache_size = inv_size * 1.5
 
192
        if entry_cache.cache_size() < recommended_min_cache_size:
 
193
            recommended_cache_size = inv_size * 2
 
194
            trace.mutter('Resizing the inventory entry cache from %d to %d',
 
195
                         entry_cache.cache_size(), recommended_cache_size)
 
196
            entry_cache.resize(recommended_cache_size)
 
197
 
 
198
    def write_inventory_to_lines(self, inv):
 
199
        """Return a list of lines with the encoded inventory."""
 
200
        return self.write_inventory(inv, None)
 
201
 
 
202
    def write_inventory_to_string(self, inv, working=False):
 
203
        """Just call write_inventory with a StringIO and return the value.
 
204
 
 
205
        :param working: If True skip history data - text_sha1, text_size,
 
206
            reference_revision, symlink_target.
 
207
        """
 
208
        sio = cStringIO.StringIO()
 
209
        self.write_inventory(inv, sio, working)
 
210
        return sio.getvalue()
 
211
 
 
212
    def write_inventory(self, inv, f, working=False):
 
213
        """Write inventory to a file.
 
214
 
 
215
        :param inv: the inventory to write.
 
216
        :param f: the file to write. (May be None if the lines are the desired
 
217
            output).
 
218
        :param working: If True skip history data - text_sha1, text_size,
 
219
            reference_revision, symlink_target.
 
220
        :return: The inventory as a list of lines.
 
221
        """
 
222
        _ensure_utf8_re()
 
223
        self._check_revisions(inv)
 
224
        output = []
 
225
        append = output.append
 
226
        self._append_inventory_root(append, inv)
 
227
        entries = inv.iter_entries()
 
228
        # Skip the root
 
229
        root_path, root_ie = entries.next()
 
230
        for path, ie in entries:
 
231
            if ie.parent_id != self.root_id:
 
232
                parent_str = ' parent_id="'
 
233
                parent_id  = _encode_and_escape(ie.parent_id)
 
234
            else:
 
235
                parent_str = ''
 
236
                parent_id  = ''
 
237
            if ie.kind == 'file':
 
238
                if ie.executable:
 
239
                    executable = ' executable="yes"'
 
240
                else:
 
241
                    executable = ''
 
242
                if not working:
 
243
                    append('<file%s file_id="%s name="%s%s%s revision="%s '
 
244
                        'text_sha1="%s" text_size="%d" />\n' % (
 
245
                        executable, _encode_and_escape(ie.file_id),
 
246
                        _encode_and_escape(ie.name), parent_str, parent_id,
 
247
                        _encode_and_escape(ie.revision), ie.text_sha1,
 
248
                        ie.text_size))
 
249
                else:
 
250
                    append('<file%s file_id="%s name="%s%s%s />\n' % (
 
251
                        executable, _encode_and_escape(ie.file_id),
 
252
                        _encode_and_escape(ie.name), parent_str, parent_id))
 
253
            elif ie.kind == 'directory':
 
254
                if not working:
 
255
                    append('<directory file_id="%s name="%s%s%s revision="%s '
 
256
                        '/>\n' % (
 
257
                        _encode_and_escape(ie.file_id),
 
258
                        _encode_and_escape(ie.name),
 
259
                        parent_str, parent_id,
 
260
                        _encode_and_escape(ie.revision)))
 
261
                else:
 
262
                    append('<directory file_id="%s name="%s%s%s />\n' % (
 
263
                        _encode_and_escape(ie.file_id),
 
264
                        _encode_and_escape(ie.name),
 
265
                        parent_str, parent_id))
 
266
            elif ie.kind == 'symlink':
 
267
                if not working:
 
268
                    append('<symlink file_id="%s name="%s%s%s revision="%s '
 
269
                        'symlink_target="%s />\n' % (
 
270
                        _encode_and_escape(ie.file_id),
 
271
                        _encode_and_escape(ie.name),
 
272
                        parent_str, parent_id,
 
273
                        _encode_and_escape(ie.revision),
 
274
                        _encode_and_escape(ie.symlink_target)))
 
275
                else:
 
276
                    append('<symlink file_id="%s name="%s%s%s />\n' % (
 
277
                        _encode_and_escape(ie.file_id),
 
278
                        _encode_and_escape(ie.name),
 
279
                        parent_str, parent_id))
 
280
            elif ie.kind == 'tree-reference':
 
281
                if ie.kind not in self.supported_kinds:
 
282
                    raise errors.UnsupportedInventoryKind(ie.kind)
 
283
                if not working:
 
284
                    append('<tree-reference file_id="%s name="%s%s%s '
 
285
                        'revision="%s reference_revision="%s />\n' % (
 
286
                        _encode_and_escape(ie.file_id),
 
287
                        _encode_and_escape(ie.name),
 
288
                        parent_str, parent_id,
 
289
                        _encode_and_escape(ie.revision),
 
290
                        _encode_and_escape(ie.reference_revision)))
 
291
                else:
 
292
                    append('<tree-reference file_id="%s name="%s%s%s />\n' % (
 
293
                        _encode_and_escape(ie.file_id),
 
294
                        _encode_and_escape(ie.name),
 
295
                        parent_str, parent_id))
 
296
            else:
 
297
                raise errors.UnsupportedInventoryKind(ie.kind)
 
298
        append('</inventory>\n')
 
299
        if f is not None:
 
300
            f.writelines(output)
 
301
        # Just to keep the cache from growing without bounds
 
302
        # but we may actually not want to do clear the cache
 
303
        #_clear_cache()
 
304
        return output
 
305
 
 
306
    def _append_inventory_root(self, append, inv):
 
307
        """Append the inventory root to output."""
38
308
        if inv.revision_id is not None:
39
 
            e.set('revision_id', inv.revision_id)
40
 
        for path, ie in inv.iter_entries():
41
 
            e.append(self._pack_entry(ie))
42
 
        return e
43
 
 
44
 
 
45
 
    def _pack_entry(self, ie):
46
 
        """Convert InventoryEntry to XML element"""
47
 
        if not InventoryEntry.versionable_kind(ie.kind):
48
 
            raise AssertionError('unsupported entry kind %s' % ie.kind)
49
 
        e = Element(ie.kind)
50
 
        e.set('name', ie.name)
51
 
        e.set('file_id', ie.file_id)
52
 
 
53
 
        if ie.text_size != None:
54
 
            e.set('text_size', '%d' % ie.text_size)
55
 
 
56
 
        for f in ['text_sha1', 'revision', 'symlink_target']:
57
 
            v = getattr(ie, f)
58
 
            if v != None:
59
 
                e.set(f, v)
60
 
 
61
 
        if ie.executable:
62
 
            e.set('executable', 'yes')
63
 
 
64
 
        # to be conservative, we don't externalize the root pointers
65
 
        # for now, leaving them as null in the xml form.  in a future
66
 
        # version it will be implied by nested elements.
67
 
        if ie.parent_id != ROOT_ID:
68
 
            assert isinstance(ie.parent_id, basestring)
69
 
            e.set('parent_id', ie.parent_id)
70
 
 
71
 
        e.tail = '\n'
72
 
 
73
 
        return e
74
 
 
 
309
            revid1 = ' revision_id="'
 
310
            revid2 = _encode_and_escape(inv.revision_id)
 
311
        else:
 
312
            revid1 = ""
 
313
            revid2 = ""
 
314
        append('<inventory format="%s"%s%s>\n' % (
 
315
            self.format_num, revid1, revid2))
 
316
        append('<directory file_id="%s name="%s revision="%s />\n' % (
 
317
            _encode_and_escape(inv.root.file_id),
 
318
            _encode_and_escape(inv.root.name),
 
319
            _encode_and_escape(inv.root.revision)))
75
320
 
76
321
    def _pack_revision(self, rev):
77
322
        """Revision object -> xml tree"""
 
323
        # For the XML format, we need to write them as Unicode rather than as
 
324
        # utf-8 strings. So that cElementTree can handle properly escaping
 
325
        # them.
 
326
        decode_utf8 = cache_utf8.decode
 
327
        revision_id = rev.revision_id
 
328
        if isinstance(revision_id, str):
 
329
            revision_id = decode_utf8(revision_id)
 
330
        format_num = self.format_num
 
331
        if self.revision_format_num is not None:
 
332
            format_num = self.revision_format_num
78
333
        root = Element('revision',
79
334
                       committer = rev.committer,
80
 
                       timestamp = '%.9f' % rev.timestamp,
81
 
                       revision_id = rev.revision_id,
 
335
                       timestamp = '%.3f' % rev.timestamp,
 
336
                       revision_id = revision_id,
82
337
                       inventory_sha1 = rev.inventory_sha1,
83
 
                       format='5',
 
338
                       format=format_num,
84
339
                       )
85
 
        if rev.timezone:
 
340
        if rev.timezone is not None:
86
341
            root.set('timezone', str(rev.timezone))
87
342
        root.text = '\n'
88
343
        msg = SubElement(root, 'message')
92
347
            pelts = SubElement(root, 'parents')
93
348
            pelts.tail = pelts.text = '\n'
94
349
            for parent_id in rev.parent_ids:
95
 
                assert isinstance(parent_id, basestring)
 
350
                _mod_revision.check_not_reserved_id(parent_id)
96
351
                p = SubElement(pelts, 'revision_ref')
97
352
                p.tail = '\n'
 
353
                if isinstance(parent_id, str):
 
354
                    parent_id = decode_utf8(parent_id)
98
355
                p.set('revision_id', parent_id)
99
356
        if rev.properties:
100
357
            self._pack_revision_properties(rev, root)
101
358
        return root
102
359
 
103
 
 
104
360
    def _pack_revision_properties(self, rev, under_element):
105
361
        top_elt = SubElement(under_element, 'properties')
106
362
        for prop_name, prop_value in sorted(rev.properties.items()):
107
 
            assert isinstance(prop_name, basestring) 
108
 
            assert isinstance(prop_value, basestring) 
109
363
            prop_elt = SubElement(top_elt, 'property')
110
364
            prop_elt.set('name', prop_name)
111
365
            prop_elt.text = prop_value
112
366
            prop_elt.tail = '\n'
113
367
        top_elt.tail = '\n'
114
368
 
115
 
 
116
 
    def _unpack_inventory(self, elt):
117
 
        """Construct from XML Element
118
 
        """
119
 
        assert elt.tag == 'inventory'
120
 
        root_id = elt.get('file_id') or ROOT_ID
 
369
    def _unpack_inventory(self, elt, revision_id=None, entry_cache=None):
 
370
        """Construct from XML Element"""
 
371
        if elt.tag != 'inventory':
 
372
            raise errors.UnexpectedInventoryFormat('Root tag is %r' % elt.tag)
121
373
        format = elt.get('format')
122
 
        if format is not None:
123
 
            if format != '5':
124
 
                raise BzrError("invalid format version %r on inventory"
125
 
                                % format)
 
374
        if format != self.format_num:
 
375
            raise errors.UnexpectedInventoryFormat('Invalid format version %r'
 
376
                                                   % format)
126
377
        revision_id = elt.get('revision_id')
127
 
        inv = Inventory(root_id, revision_id=revision_id)
 
378
        if revision_id is not None:
 
379
            revision_id = cache_utf8.encode(revision_id)
 
380
        inv = inventory.Inventory(root_id=None, revision_id=revision_id)
128
381
        for e in elt:
129
 
            ie = self._unpack_entry(e)
130
 
            if ie.parent_id == ROOT_ID:
131
 
                ie.parent_id = root_id
 
382
            ie = self._unpack_entry(e, entry_cache=entry_cache)
132
383
            inv.add(ie)
 
384
        self._check_cache_size(len(inv), entry_cache)
133
385
        return inv
134
386
 
135
 
 
136
 
    def _unpack_entry(self, elt):
 
387
    def _unpack_entry(self, elt, entry_cache=None):
 
388
        elt_get = elt.get
 
389
        file_id = elt_get('file_id')
 
390
        revision = elt_get('revision')
 
391
        # Check and see if we have already unpacked this exact entry
 
392
        # Some timings for "repo.revision_trees(last_100_revs)"
 
393
        #               bzr     mysql
 
394
        #   unmodified  4.1s    40.8s
 
395
        #   using lru   3.5s
 
396
        #   using fifo  2.83s   29.1s
 
397
        #   lru._cache  2.8s
 
398
        #   dict        2.75s   26.8s
 
399
        #   inv.add     2.5s    26.0s
 
400
        #   no_copy     2.00s   20.5s
 
401
        #   no_c,dict   1.95s   18.0s
 
402
        # Note that a cache of 10k nodes is more than sufficient to hold all of
 
403
        # the inventory for the last 100 revs for bzr, but not for mysql (20k
 
404
        # is enough for mysql, which saves the same 2s as using a dict)
 
405
 
 
406
        # Breakdown of mysql using time.clock()
 
407
        #   4.1s    2 calls to element.get for file_id, revision_id
 
408
        #   4.5s    cache_hit lookup
 
409
        #   7.1s    InventoryFile.copy()
 
410
        #   2.4s    InventoryDirectory.copy()
 
411
        #   0.4s    decoding unique entries
 
412
        #   1.6s    decoding entries after FIFO fills up
 
413
        #   0.8s    Adding nodes to FIFO (including flushes)
 
414
        #   0.1s    cache miss lookups
 
415
        # Using an LRU cache
 
416
        #   4.1s    2 calls to element.get for file_id, revision_id
 
417
        #   9.9s    cache_hit lookup
 
418
        #   10.8s   InventoryEntry.copy()
 
419
        #   0.3s    cache miss lookus
 
420
        #   1.2s    decoding entries
 
421
        #   1.0s    adding nodes to LRU
 
422
        if entry_cache is not None and revision is not None:
 
423
            key = (file_id, revision)
 
424
            try:
 
425
                # We copy it, because some operatations may mutate it
 
426
                cached_ie = entry_cache[key]
 
427
            except KeyError:
 
428
                pass
 
429
            else:
 
430
                # Only copying directory entries drops us 2.85s => 2.35s
 
431
                # if cached_ie.kind == 'directory':
 
432
                #     return cached_ie.copy()
 
433
                # return cached_ie
 
434
                return cached_ie.copy()
 
435
 
137
436
        kind = elt.tag
138
437
        if not InventoryEntry.versionable_kind(kind):
139
438
            raise AssertionError('unsupported entry kind %s' % kind)
140
439
 
141
 
        parent_id = elt.get('parent_id')
142
 
        if parent_id == None:
143
 
            parent_id = ROOT_ID
 
440
        get_cached = _get_utf8_or_ascii
 
441
 
 
442
        file_id = get_cached(file_id)
 
443
        if revision is not None:
 
444
            revision = get_cached(revision)
 
445
        parent_id = elt_get('parent_id')
 
446
        if parent_id is not None:
 
447
            parent_id = get_cached(parent_id)
144
448
 
145
449
        if kind == 'directory':
146
 
            ie = inventory.InventoryDirectory(elt.get('file_id'),
147
 
                                              elt.get('name'),
 
450
            ie = inventory.InventoryDirectory(file_id,
 
451
                                              elt_get('name'),
148
452
                                              parent_id)
149
453
        elif kind == 'file':
150
 
            ie = inventory.InventoryFile(elt.get('file_id'),
151
 
                                         elt.get('name'),
 
454
            ie = inventory.InventoryFile(file_id,
 
455
                                         elt_get('name'),
152
456
                                         parent_id)
153
 
            ie.text_sha1 = elt.get('text_sha1')
154
 
            if elt.get('executable') == 'yes':
 
457
            ie.text_sha1 = elt_get('text_sha1')
 
458
            if elt_get('executable') == 'yes':
155
459
                ie.executable = True
156
 
            v = elt.get('text_size')
 
460
            v = elt_get('text_size')
157
461
            ie.text_size = v and int(v)
158
462
        elif kind == 'symlink':
159
 
            ie = inventory.InventoryLink(elt.get('file_id'),
160
 
                                         elt.get('name'),
 
463
            ie = inventory.InventoryLink(file_id,
 
464
                                         elt_get('name'),
161
465
                                         parent_id)
162
 
            ie.symlink_target = elt.get('symlink_target')
 
466
            ie.symlink_target = elt_get('symlink_target')
163
467
        else:
164
 
            raise BzrError("unknown kind %r" % kind)
165
 
        ie.revision = elt.get('revision')
 
468
            raise errors.UnsupportedInventoryKind(kind)
 
469
        ie.revision = revision
 
470
        if revision is not None and entry_cache is not None:
 
471
            # We cache a copy() because callers like to mutate objects, and
 
472
            # that would cause the item in cache to mutate as well.
 
473
            # This has a small effect on many-inventory performance, because
 
474
            # the majority fraction is spent in cache hits, not misses.
 
475
            entry_cache[key] = ie.copy()
166
476
 
167
477
        return ie
168
478
 
169
 
 
170
479
    def _unpack_revision(self, elt):
171
480
        """XML Element -> Revision object"""
172
 
        assert elt.tag == 'revision'
173
481
        format = elt.get('format')
 
482
        format_num = self.format_num
 
483
        if self.revision_format_num is not None:
 
484
            format_num = self.revision_format_num
174
485
        if format is not None:
175
 
            if format != '5':
176
 
                raise BzrError("invalid format version %r on inventory"
 
486
            if format != format_num:
 
487
                raise BzrError("invalid format version %r on revision"
177
488
                                % format)
 
489
        get_cached = _get_utf8_or_ascii
178
490
        rev = Revision(committer = elt.get('committer'),
179
491
                       timestamp = float(elt.get('timestamp')),
180
 
                       revision_id = elt.get('revision_id'),
 
492
                       revision_id = get_cached(elt.get('revision_id')),
181
493
                       inventory_sha1 = elt.get('inventory_sha1')
182
494
                       )
183
495
        parents = elt.find('parents') or []
184
496
        for p in parents:
185
 
            assert p.tag == 'revision_ref', \
186
 
                   "bad parent node tag %r" % p.tag
187
 
            rev.parent_ids.append(p.get('revision_id'))
 
497
            rev.parent_ids.append(get_cached(p.get('revision_id')))
188
498
        self._unpack_revision_properties(elt, rev)
189
499
        v = elt.get('timezone')
190
 
        rev.timezone = v and int(v)
 
500
        if v is None:
 
501
            rev.timezone = 0
 
502
        else:
 
503
            rev.timezone = int(v)
191
504
        rev.message = elt.findtext('message') # text of <message>
192
505
        return rev
193
506
 
194
 
 
195
507
    def _unpack_revision_properties(self, elt, rev):
196
508
        """Unpack properties onto a revision."""
197
509
        props_elt = elt.find('properties')
198
 
        assert len(rev.properties) == 0
199
510
        if not props_elt:
200
511
            return
201
512
        for prop_elt in props_elt:
202
 
            assert prop_elt.tag == 'property', \
203
 
                "bad tag under properties list: %r" % p.tag
 
513
            if prop_elt.tag != 'property':
 
514
                raise AssertionError(
 
515
                    "bad tag under properties list: %r" % prop_elt.tag)
204
516
            name = prop_elt.get('name')
205
517
            value = prop_elt.text
206
 
            assert name not in rev.properties, \
207
 
                "repeated property %r" % p.name
 
518
            # If a property had an empty value ('') cElementTree reads
 
519
            # that back as None, convert it back to '', so that all
 
520
            # properties have string values
 
521
            if value is None:
 
522
                value = ''
 
523
            if name in rev.properties:
 
524
                raise AssertionError("repeated property %r" % name)
208
525
            rev.properties[name] = value
209
526
 
210
527
 
211
 
serializer_v5 = Serializer_v5()
 
528
serializer_v8 = Serializer_v8()