Bug 523670

Summary: "useentrygroupdefault" parameter was removed partially from 3.4 code
Product: [Community] Bugzilla Reporter: Noura El hawary <nelhawar>
Component: Bugzilla GeneralAssignee: Noura El hawary <nelhawar>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: low    
Version: 3.4   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-10-20 09:30:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 514793    
Attachments:
Description Flags
v1 of patch to remove useentrygroupdefault parameter dkl: review+

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