Bug 538149 - create new content provider with invalid values completes without any error if click save a second time
Summary: create new content provider with invalid values completes without any error i...
Keywords:
Status: CLOSED WORKSFORME
Alias: None
Product: RHQ Project
Classification: Other
Component: Configuration
Version: unspecified
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: RHQ Project Maintainer
QA Contact:
URL:
Whiteboard:
: 536883 (view as bug list)
Depends On:
Blocks: RHQ_CONT_REPO
TreeView+ depends on / blocked
 
Reported: 2009-11-17 18:39 UTC by Preethi Thomas
Modified: 2010-03-29 17:34 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-03-29 17:34:30 UTC
Embargoed:


Attachments (Terms of Use)

Description Preethi Thomas 2009-11-17 18:39:53 UTC
Description of problem:

Create new content provider with invalid values completes without any error if click save a second time
Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.Administration->Content->Content Provider->Create New-> RHN Hosted Source
2.Enter Values for all the required fields
3.Enter invalid location (htp:)
5. Click save
6. See the error.
7. Click on save again
Actual results:

The content provider is created without any errors

Expected results:
Content provider should fail to create.


Additional info:

Comment 1 Jay Dobies 2009-12-04 20:12:33 UTC
I think it's related, but the new information entered is never sent to the content provider. A slight variation on the above reproduction:

1.Administration->Content->Content Provider->Create New-> RHN Hosted Source
2.Enter Values for all the required fields
3.Enter invalid location (htp:)
5. Click save
6. See the error.
** 7. Enter correct data ***
8. Click on save again

Here I expected the provider to be created without any errors, which it did. However, when I went to use the provider it continued to use the bad property values.

Comment 2 Jay Dobies 2009-12-04 20:14:27 UTC
*** Bug 536883 has been marked as a duplicate of this bug. ***

Comment 3 Jay Dobies 2009-12-07 20:56:15 UTC
commit	2d66e832766c3ec234f08bb7105d5b6dad57fe9e
tree	2b305d3dbd08303b4d2f630109bb10bcb180b5b5

ContentProviderManager.java

If the provider does not start correctly, we need to make sure it's removed from the cache. Otherwise, future calls to start (which occur when the user attempts to correct the failed start errors) will simply return assuming it's started.

Comment 4 Corey Welton 2010-03-29 17:32:27 UTC
Mass-move to ON_QA

Comment 5 Corey Welton 2010-03-29 17:34:30 UTC
QA Closing - essentially untestable now.


Note You need to log in before you can comment on or make changes to this bug.