VersionControl API inconsistency

[athomas]

trac.versioncontrol.api.Repository advertises get_node() and has_node() as accepting None as the default rev.

The underlying SVN bindings used by svn_fs.SubversionRepository do not appear to accept None as a rev, and SubversionRepository.has_node() does not have logic to deal with this (get_node() does).

Additionally, SubversionRepository.has_node() accepts a pool argument, while get_node() doesn't. I'm not sure whether this is intentional, but it seemed a bit odd.

[athomas]

Patch fixing has_node() in the VC layer

[athomas]

Also, this is a bit odd:

rev = repo.get_youngest_rev()
self.env.log.debug(type(rev))

Logs this:

2007-01-24 17:43:45,977 Trac[__init__] DEBUG: <type 'unicode'>

[cboos]

Yes, it's a bit tricky: since 0.10.3, the get_youngest_rev is now done on the cache repos, not on the svn repos (see r4303). Therefore we now have unicode revs. We probably need to normalize the rev to an int in SubversionRepository.get_youngest_rev_in_cache.

[cboos]

See #4830.