Hi,
I've found a bug in Integration.pm code, when I was playing around with
case insensitive comparison of 'bug' and 'status' keywords.
As a bug tracking system we use Bugzilla and it requires to have status
resolution in upper-case letters. So in glue.conf I switched on
parameter  "resolution_status_case_sensitive_verification => 1".
BUT after committing with log message "...status 111: ReSOLVeD FIXeD"
SmBug verification in precommit phase has passed successfully, but
post-commit hook has returned an error message and status in bugzilla
didn't change.

After little bit of debugging I found, that in Intergation.pm on line
which should verify the status change parameter
resolution_status_case_sensitive_verification is badly referred.

my ( $resolution_change_retval, $current_resolution_state ) =
$self->bugtracker()->integration_bug_resolution_change_is_valid(
$resolution_id, $self->request()->{ resolution_ids }->{ $resolution_id }->{
status }, $self->request()->{ resolution_ids }->{ $resolution_id }->{
resolution }, $self->request()->{ policies }->{
resolution_status_case_sensitive_verification } );

=>

my ( $resolution_change_retval, $current_resolution_state ) =
$self->bugtracker()->integration_bug_resolution_change_is_valid(
$resolution_id, $self->request()->{ resolution_ids }->{ $resolution_id }->{
status }, $self->request()->{ resolution_ids }->{ $resolution_id }->{
resolution }, $self->request()->{ policies }->{ resolution_template }->{
resolution_status_case_sensitive_verification } );



I didn't know where to place a bug I've found so this was the suitable
only place.

Petr

_______________________________________________
scmbug-users mailing list
[email protected]
http://lists.mkgnu.net/cgi-bin/mailman/listinfo/scmbug-users

Reply via email to