Bug 815679

Summary:	software error shows when searching certs with old url
Product:	[Retired] Red Hat Hardware Certification Program	Reporter:	Wei Shen <wshen>
Component:	Hardware Catalog	Assignee:	Wei Shen <wshen>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	5	CC:	hwcert-catalog, junwang, rlandry, tfu
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-06-06 04:22:42 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:
Bug Depends On:
Bug Blocks:	733411

Description Wei Shen 2012-04-24 09:01:32 UTC

Description of problem:
Dell vendor found this bug. 

https://hardware.redhat.com/list.cgi?product=Red+Hat+Hardware+Certification&search_type=&component=Red+Hat+Enterprise+Linux&version=5&bug_status=Certified&devel_whiteboard=dell&devel_whiteboard_type=allwordssubstr&quicksearch=&internal_whiteboard=Server

This url will result in software error:
can't use an undefined value as an ARRAY reference at Bugzilla/Search.pm line 179.

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


How reproducible:
Every time

Steps to Reproduce:
1. click the url
2.
3.
  
Actual results:
  software error

Expected results:
   no error and certs returned

Additional info:

Comment 1 Wei Shen 2012-04-24 09:05:56 UTC

Created attachment 579794 [details]
the patch

Comment 2 Rob Landry 2012-04-24 16:57:00 UTC

Should we open a second bug for the cf_* remappings?

Comment 3 Wei Shen 2012-04-25 03:18:58 UTC

the root cause of this bug is some syntax errors in code, I don't think other cf_* are used in old code, so we should be ok

Comment 4 Rob Landry 2012-04-25 13:55:07 UTC

The cf_* are fields that exist now that didn't before.  Old links that refer to the fields without the cf_* will be ignored by the code right?  If this is correct should we open a BZ for the purpose of having the code correctly update the none cf_* field name to cf_* field name so that the desired result can be returned?

Comment 5 Tony Fu 2012-04-25 23:32:58 UTC

Comment on attachment 579794 [details]
the patch

the patch looks good for me

Comment 6 Wei Shen 2012-04-26 06:35:33 UTC

we can add a new bug for remapping, for now, I have verified this patch on web2

Comment 7 Wei Shen 2012-05-17 07:11:08 UTC

verified on partner

Comment 8 Tony Fu 2012-06-06 04:22:42 UTC

Verified on live.

Closing this bug.