[prev in list] [next in list] [prev in thread] [next in thread] 

List:       tortoisesvn-users
Subject:    RE: Re: TSVN 1.7.7: Invalid storing bugtraq:logarex while using simplified dialog
From:       Andriy Kvasnytsya <celebrin () inbox ! ru>
Date:       2012-05-25 7:13:45
Message-ID: 869212204.16320.1337930025299.JavaMail.httpd () localhost
[Download RAW message or body]

> The log regex *is* a two line string: the first line is the "message 
> part expression" which must catch the whole bug expression, e.g. "issue 
> #233". The second line then catches only the issue id itself, e.g., "233".

Hi Stefan,

Obviously, it is. But as far as I understand, it also supports one line mode - with \
only bug id regex.  At least my experience (it was working before mentioned change, \
and after manual tweak - works after) and documentation at \
http://tortoisesvn.net/docs/release/TortoiseSVN_sk/tsvn-dug-bugtracker.html are \
proving that.

That's very nice that simplified UI allows user to modify settings without precisely \
picking complicated and unobvious property names. But if simple edit-ok (without any \
modifications) breaks things that had already worked, that's not very nice. Actually, \
it's frustrating, because I've spend half an hour trying to understand what's wrong \
with my regexes (pretty trivial, that makes attempt to find nonexisting bugs in them \
even harder), or did something changed in way how Tortoise SVN is handling them... \
Just to find out it stores regexes in different way than earlier, or actually tries \
to force me to use "officially recommened" way of using log regexes without actually \
telling me.

I can suggest:
1) add some warning in case there is "bug-id" regex and no "message part" regex, \
pointing to documentation 2) store it as one and single regex in property, if user \
still want's to continue (as it's still allowed in documentation)

Or:
1) Don't allow user to finish dialogue without "message part", again pointing to \
documentation 2) Mention in documentation (and optionally in error message) that user \
have to use "Advanced" edit mode to let this work with single regex.

In it's current state applications behavior is unobvious and extremely non-user \
friendly.

Sorry for arguing with you about this beautiful tool, but I think both of us want to \
make it even better.

BR,
A.K.

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=2964990

To unsubscribe from this discussion, e-mail: \
[users-unsubscribe@tortoisesvn.tigris.org].


[prev in list] [next in list] [prev in thread] [next in thread]