Bug 961594

Summary:	Organizations: field when editing an org description allows more text to be entered than is allowed.
Product:	Red Hat Satellite	Reporter:	Corey Welton <cwelton>
Component:	WebUI	Assignee:	Mike McCune <mmccune>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Katello QA List <katello-qa-list>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	Nightly	CC:	bbuckingham, mmccune, omaciel
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:	2014-04-24 17:07:59 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:

Description Corey Welton 2013-05-10 03:07:03 UTC

Description of problem:
When editing the description of an Org, user can enter long descriptions in the field which outrun the max length set in db.

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


How reproducible:


Steps to Reproduce:
1.  Organizations > ACME_Corporation 
2.  Attempt to edit the description of this org, using the following value:
"This is a very long description.  This is a very long description.  This is a very long description.  This is a very long description.  This is a very long description.  This is a very long description.  This is a very long description.  This is a very long description.  "
3. Save
  
Actual results:
"Validation Failed:
Description cannot contain more than 255 characters"

description is not saved.

Expected results:
User should not be able to enter more data than is allowed by the db.

Additional info:

Comment 1 manjiri tapaswi 2013-06-05 15:41:45 UTC

Set maxlength to restrict data entered in org description textarea as defined in the db

https://github.com/Katello/katello/pull/2429

Added support for maxlength attribute for jeditable textarea 

https://github.com/ui-alchemy/ui_alchemy-rails/pull/160

Comment 2 manjiri tapaswi 2013-06-05 21:03:46 UTC

updated PR: https://github.com/Katello/katello/pull/2432

Comment 4 manjiri tapaswi 2013-06-13 17:55:19 UTC

Latest PR: https://github.com/Katello/katello/pull/2488

Comment 5 manjiri tapaswi 2013-06-14 15:38:16 UTC

Commit: fc01e13

Comment 7 Corey Welton 2013-10-14 18:40:35 UTC

QA Verified in Satellite-6.0.2-RHEL-6-20131011.0

Comment 9 Bryan Kearney 2014-04-24 17:07:59 UTC

This was verified and delivered with MDP2. Closing it out.