Bug 715422

Summary:	Opening sync status page directly from URL gives TypeError
Product:	Red Hat Satellite	Reporter:	Jeff Weiss <jweiss>
Component:	WebUI	Assignee:	Shannon Hughes <shughes>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Katello QA List <katello-qa-list>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.0.0	CC:	cwelton, dajohnso
Target Milestone:	Unspecified	Keywords:	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:	2012-08-22 17:47:13 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:
Bug Blocks:	747354

Description Jeff Weiss 2011-06-22 20:48:27 UTC

Description of problem:


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


How reproducible:


Steps to Reproduce:
1. Log out
2. Open url  http://[host]:3000/sync_management/index
3.
  
Actual results:
TypeError in Sync managementController#index

can't convert nil into String
Rails.root: /usr/lib/katello

Application Trace | Framework Trace | Full Trace
lib/resources/pulp.rb:211:in `+'
lib/resources/pulp.rb:211:in `sync_history'
app/models/glue/pulp/repo.rb:148:in `_get_most_recent_sync_status'
app/models/glue/pulp/repo.rb:102:in `sync_status'
app/controllers/sync_management_controller.rb:60:in `status'
app/controllers/application_controller.rb:185:in `require_org'
Request

Parameters:

None
Show session dump

Show env dump

Expected results:
Gracefully handle error - if login required, redirect to login page.

Additional info:

Comment 1 Shannon Hughes 2011-07-12 16:42:41 UTC

good catch and very odd issue. looks like 'status' is a reserved method name for rails. had to change to 'sync_status' and then update routes and javascript polling to use new naming covention. apparently status was getting called for all index requests. 

ea28466..f4ab09b  master -> master

Comment 2 Corey Welton 2011-08-05 15:52:35 UTC

Verified.