On 4/7/2014 8:59 PM, Andy Bradford wrote:
Thus said Andy Goth on Mon, 07 Apr 2014 14:33:10 -0500:
Those look fine, but did you really intend to reject all requests
for UUID abbreviations shorter than four characters?
Yes, it was intentional. It was not my intention to change the length
at which short UUIDs were supported, only to prevent an existing
ticket from masking any other potential collisions.
Yeah, no need to change. Matching on very short prefixes isn't useful.
Do you think the error message would be better if worded to indicate
that a longer UUID is required?
Yes.
To be fair, there is still one type of UUID that will work at a
shorter length than 4; events currently do not go through collision
detection, so it's possible to view an event with a length shorter
than 4:
http://fossil.bradfords.org:8080/info/a
Should I also add event collision handling to this branch?
I do prefer consistency, but we're going to need to think more about
whether we want to break something which used to work just for the sake
of consistency.
--
Andy Goth | <andrew.m.goth/at/gmail/dot/com>
_______________________________________________
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users
