Re: Recent messages duplicate bugtaq:message

On 18.12.2009 23:53, Dominik Guder wrote:
> Hi,
>
> I yould not find anything in mailing list archive about this. If this
> was discussed before, please point me to the correct location.
>
> I enabled bugtraq integration a long time ago with following configuration:
>
> bugtraq:message: Issue: ABCD-%BUGID%
> bugtraq:number: true
> bugtrq:url: http://someurl.com/view.php?id=%BUGID%
>
> I can't remember when I discovered following issues maybe in 1.5 but for
> sure it is still in 1.6.6:
> a) Duplicate bugtraq:message:
> - write a new message "Some message" and add an issue number, e.g. 1234
> - "Issue: ABCD-1234" will be added to log entry
> - Resulting log message:
> Some message
> Issue: ABCD-1234
> - so far so good
> - do another commit and select the above used log message from recent
> messages
> - in log message window you will see following:
> Some message
> Issue: ABCD-1234
> - in addition the number 1234 is filled into issue number box
> - when you commit now you will get following (maybe with or without
> empty line):
> Some message
> Issue: ABCD-1234
>
> Issue: ABCD-1234
> - I definifely don't want to have this double issue information in log.
> - IMHO before this issue occured the bugtraq:message was stripped when
> using recent messages.

I can't reproduce this with the steps you listed above. There's a check
in the commit dialog code to prevent exactly this situation: it compares
the issue ID you've entered in the box at the top right with the one it
finds in the entered log message - if those are the same it doesn't add
the issue line to the message anymore. So you'll only get one line, not two.
Is there maybe a step you've been missing from the list above which
would trigger this bug?

> b) removed commas from number list:
> bugtraq:number: true enforces to use only numbers and separate them with
> comma, e.g. 1234,1235,1236
> - use this list for a single commit
> - select this commit message from recent messages
> - the number box will be filled with this three numbers, but separated
> with whitespace not with comma anymore. So you will get an error since
> "bugtraq:number: true" is violated

Same here, can't reproduce this.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2431682
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].