Bug 1018069

Summary: Removing rule parameters raised undefined method error
Product: Red Hat Satellite Reporter: Sachin Ghai <sghai>
Component: WebUIAssignee: Mike McCune <mmccune>
Status: CLOSED CURRENTRELEASE QA Contact: Sachin Ghai <sghai>
Severity: medium Docs Contact:
Priority: high    
Version: 6.0.2CC: adujicek, jmontleo, mmccune
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-04-24 17:09:54 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
undefined method error while removing filter rule's parameters
none
undefined method error while removing package-group rule
none
undefined method error while removing filter rule's parameters (snap7 compose2)
none
success on removing rule parameters (snap8) none

Description Sachin Ghai 2013-10-11 06:53:08 UTC
Created attachment 810881 [details]
undefined method error while removing filter rule's parameters

Description of problem:
I created a filter of type errata and selected errata-id ("RHEA-2012:3642") and saved the rule. Later I tried to remove it and then UI raised following error:

    undefined method `join' for nil:NilClass (NoMethodError)
    Click here for more details.

Version-Release number of selected component (if applicable):
snap4 compose4 (mdp2)

katello-1.4.6-21.el6sat.noarch

How reproducible:
always

Steps to Reproduce:
1. create a cv definition
2. select the product/repo from content-tab
3. create a cv filter
4. select the repo from repo tab
5. Add rule and select filter type 'Errata'
6. select the 'Errata-id"..
7. Use auto-completion
8. save the rule
9. select the checkbox and click 'Remove' to delete the added rule


Actual results:
undefined method `join' for nil:NilClass (NoMethodError)
    Click here for more details.


Expected results:
Rule parameters should be removed successfully

Additional info:

Comment 2 Sachin Ghai 2013-10-11 07:04:08 UTC
Found same issue with package groups.

Comment 3 Sachin Ghai 2013-10-11 07:05:50 UTC
Created attachment 810884 [details]
undefined method error while removing package-group rule

Comment 4 manjiri tapaswi 2013-10-18 19:52:46 UTC
PR: https://github.com/Katello/katello/pull/3214

Comment 5 Og Maciel 2013-10-21 18:37:37 UTC
*** Bug 957650 has been marked as a duplicate of this bug. ***

Comment 6 manjiri tapaswi 2013-10-24 15:23:28 UTC
Commit: c5f7a8f

Comment 8 Sachin Ghai 2013-10-28 07:43:01 UTC
Still reproducible with Snap7 compose2:

signo-katello-0.0.23-2.el6sat.noarch
katello-1.4.6-49.el6sat.noarch
katello-qpid-client-key-pair-1.0-1.noarch
katello-cli-common-1.4.3-27.el6sat.noarch
katello-foreman-all-1.4.6-49.el6sat.noarch
katello-candlepin-cert-key-pair-1.0-1.noarch
ruby193-rubygem-katello_api-0.0.3-4.el6sat.noarch
katello-selinux-1.4.4-4.el6sat.noarch
pulp-katello-plugins-0.2-1.el6sat.noarch
katello-configure-foreman-1.4.7-7.el6sat.noarch
katello-certs-tools-1.4.4-1.el6sat.noarch
katello-qpid-broker-key-pair-1.0-1.noarch
katello-glue-elasticsearch-1.4.6-49.el6sat.noarch
katello-glue-pulp-1.4.6-49.el6sat.noarch
ruby193-rubygem-foreman-katello-engine-0.0.17-7.el6sat.noarch
katello-configure-1.4.7-7.el6sat.noarch
katello-glue-candlepin-1.4.6-49.el6sat.noarch
katello-cli-1.4.3-27.el6sat.noarch
katello-all-1.4.6-49.el6sat.noarch
katello-common-1.4.6-49.el6sat.noarch
ruby193-rubygem-katello-foreman-engine-0.0.12-3.el6sat.noarch

Comment 9 Sachin Ghai 2013-10-28 07:48:05 UTC
Created attachment 816712 [details]
undefined method error while removing filter rule's parameters (snap7 compose2)

Comment 12 Sachin Ghai 2013-10-31 08:44:21 UTC
Verified with Snap8 (MDP2)


katello-qpid-client-key-pair-1.0-1.noarch
ruby193-rubygem-katello_api-0.0.3-4.el6sat.noarch
katello-glue-pulp-1.4.6-55.el6sat.noarch
katello-certs-tools-1.4.4-1.el6sat.noarch
katello-foreman-all-1.4.6-55.el6sat.noarch
katello-candlepin-cert-key-pair-1.0-1.noarch
katello-selinux-1.4.4-4.el6sat.noarch
katello-1.4.6-55.el6sat.noarch
katello-cli-1.4.3-28.el6sat.noarch
katello-qpid-broker-key-pair-1.0-1.noarch
katello-glue-elasticsearch-1.4.6-55.el6sat.noarch
ruby193-rubygem-katello-foreman-engine-0.0.12-4.el6sat.noarch
ruby193-rubygem-foreman-katello-engine-0.0.17-7.el6sat.noarch
pulp-katello-plugins-0.2-1.el6sat.noarch
katello-glue-candlepin-1.4.6-55.el6sat.noarch
katello-configure-foreman-1.4.7-7.el6sat.noarch
katello-cli-common-1.4.3-28.el6sat.noarch
katello-all-1.4.6-55.el6sat.noarch
katello-common-1.4.6-55.el6sat.noarch
katello-configure-1.4.7-7.el6sat.noarch


The reported issue is fixed now I can remove the rule parameters successfully.

Comment 13 Sachin Ghai 2013-10-31 08:45:32 UTC
Created attachment 817756 [details]
success on removing rule parameters (snap8)

Comment 15 Bryan Kearney 2014-04-24 17:09:54 UTC
This was verified and delivered with MDP2. Closing it out.

Comment 16 Bryan Kearney 2014-04-24 17:11:17 UTC
This was delivered and verified with MDP2. Closing the bug.