Bug 734073

Summary: dashboard name max length should be restricted on client side (GUI) and should revert original name on error case
Product: [Other] RHQ Project Reporter: Jeeva Kandasamy <jkandasa>
Component: Core UIAssignee: Ian Springer <ian.springer>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: medium Docs Contact:
Priority: medium    
Version: 4.1CC: ccrouch, hrupp, ian.springer
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 4.2 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-07 19:20:10 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: 730796    
Attachments:
Description Flags
screen shot
none
back-end log file from server none

Description Jeeva Kandasamy 2011-08-29 11:36:23 UTC
Created attachment 520359 [details]
screen shot

Description of problem:
Seems dashboard name max length supported on Database is 200 chars. There is no restriction on client side (GUI). If we enter more than 200 chars throws exception on server backed and error message on GUI, but it is not much relevant for this error. GWT shows new name until navigate to another menu!

Version-Release number of selected component (if applicable):
Version: 4.1.0-SNAPSHOT
Build Number: a98c893
GWT Version: 2.0.4
SmartGWT Version: 2.4

Browser: Firefox 3.6.18
OS: RHEL 6.1 X86_64

How reproducible:
100%

Steps to Reproduce:
1.Navigate to dashboard
2.Select a dashboard and go to edit mode
3.Change the name, new name length should be more than 200 chars
4.Click on 'View mode'
5.Throws exception on back-end (valid), but new name is showing on dashboard until navigate to some other menu.  
  
Actual results:
Throws exception on backend and exception message on GUI (not much user friendly), GWT holds new name until navigate to another menu.

Expected results:
Should be validated on client side itself, do not load server with huge error trace. On GUI side message should be more specific for this error and old name should be retained on GWT.

Additional info: Screen shot and log files are attached.

Comment 1 Jeeva Kandasamy 2011-08-29 11:37:07 UTC
Created attachment 520360 [details]
back-end log file from server

Comment 2 Ian Springer 2011-10-21 17:41:58 UTC
Fixed - [master e91ab5c] (http://git.fedorahosted.org/git/?p=rhq/rhq.git;a=commitdiff;h=e91ab5c).

Comment 3 Mike Foley 2011-10-21 21:16:51 UTC
build #607

Comment 4 Mike Foley 2012-02-07 19:20:10 UTC
changing status of VERIFIED BZs for JON 2.4.2 and JON 3.0 to CLOSED/CURRENTRELEASE