Bug 523670 - "useentrygroupdefault" parameter was removed partially from 3.4 code
Summary: "useentrygroupdefault" parameter was removed partially from 3.4 code
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Bugzilla
Classification: Community
Component: Bugzilla General
Version: 3.4
Hardware: All
OS: All
low
medium
Target Milestone: ---
Assignee: Noura El hawary
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: RHBZ34UpgradeTracker
TreeView+ depends on / blocked
 
Reported: 2009-09-16 12:29 UTC by Noura El hawary
Modified: 2025-10-16 23:26 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-10-20 09:30:25 UTC
Embargoed:


Attachments (Terms of Use)
v1 of patch to remove useentrygroupdefault parameter (2.05 KB, patch)
2009-10-18 12:14 UTC, Noura El hawary
dkl: review+
Details | Diff

Description Noura El hawary 2009-09-16 12:29:02 UTC
running testcase 205_selenium_default_groups.t produced errors

errors were because parameter "useentrygroupdefault" was removed from our 3.4 code, checking upstream bugzilla, this parameter was supposed to be removed in 3.6, all details can be found here:
https://bugzilla.mozilla.org/show_bug.cgi?id=478972

the patch from the above bug was partially applied to our 3.4 branch, for example the parameter still exist in Bugzilla/User.pm

testopia testcase:
https://testopia.devel.redhat.com/bugzilla/tr_show_case.cgi?case_id=12145

Comment 1 Noura El hawary 2009-10-18 12:14:49 UTC
Created attachment 365153 [details]
v1 of patch to remove useentrygroupdefault parameter

Hi Dave,

as mentioned in the description of this bug, this patch should remove the parameter useentrydefaultgroup completely from the code to make the testcase 
t/205_selenium_default_groups.t pass after removing the tests related to that parameter from the testcase. please review and let me know what you think.

Thanks,
Noura

Comment 2 David Lawrence 2009-10-19 16:03:09 UTC
Comment on attachment 365153 [details]
v1 of patch to remove useentrygroupdefault parameter

Looks good.

Comment 3 Noura El hawary 2009-10-20 09:30:25 UTC
Thanks for the review Dave. Patch committed to svn


Note You need to log in before you can comment on or make changes to this bug.