Oliver, which bug does this work refer to ?
On Sun, 2007-12-23 at 11:57 +0100, Oliver Schäfer wrote: > Hello Kristis, > > I'm referring to Integration.pm.in (process_activity_tag). I think we > should call integration_tag_exists for $TAG_OPERATION_DELETE too. > > integration_tag_exists does all the checks for addition and removal > of a tag. So we could remove redundant checks in integration_add_tag/ > integration_delete_tag in the future (e.g. check for existence of the > product), or not? This has an impact on other bugtrackers too, and > it's just redundant, so we might don't change these methods for now. > > At least we should do integration_tag_exists in TAG_OPERATION_DELETE, > see attached fix (tested with SVN+Bugzilla). > > > -- > Oliver > > > >
signature.asc
Description: This is a digitally signed message part
_______________________________________________ scmbug-users mailing list [email protected] http://lists.mkgnu.net/cgi-bin/mailman/listinfo/scmbug-users
