Bug 755673 - Adding new drift: includes/excludes - Path property required?
Summary: Adding new drift: includes/excludes - Path property required?
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.2
Hardware: All
OS: All
medium
low
Target Milestone: ---
: ---,RHQ 4.3.0
Assignee: Jay Shaughnessy
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks: 707225 jon30-sprint10, rhq43-sprint10
TreeView+ depends on / blocked
 
Reported: 2011-11-21 18:08 UTC by Libor Zoubek
Modified: 2015-11-02 00:42 UTC (History)
3 users (show)

Fixed In Version: 4.3
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-07 19:26:42 UTC
Embargoed:


Attachments (Terms of Use)

Description Libor Zoubek 2011-11-21 18:08:26 UTC
This one is just a little usability enhancement

Description of problem: When defining includes/excludes in new drift there is no need to require path property not to be empty. For case, when I have base directory set to lets say /opt/drifts and I want to only specify a few include masks I am forced to put '.' into Path field. 


Version-Release number of selected component (if applicable):
JON 3.0.0.CR2

How reproducible: always


Steps to Reproduce:
1. Create new drift from File-System Template
2. Define some base dir
3. Specify you want to include only *.txt files within base dir
  
Actual results: you are forced to insert '.' symbol to 'Path' property 


Expected results: It would be more simple if 'Path' property could be empty and would mean, include given mask within base directory


Additional info:

Comment 1 Charles Crouch 2011-11-29 02:42:37 UTC
(8:29:08 PM) ccrouch: jsanda: this seems a reasonable request: https://bugzilla.redhat.com/show_bug.cgi?id=755673, but for jon3 this should be a note in the docs right?
(8:29:34 PM) jsanda: looking...
(8:30:44 PM) jsanda: definitely a reasonable request
(8:31:01 PM) jsanda: just trying to remember if there was a good reason we required the field to be non-empty
(8:31:24 PM) jsanda: whatever the reason, must not have been good enough because i can't think of it :-)

Comment 2 Jay Shaughnessy 2011-12-09 19:46:05 UTC
master commit 84c83f8760f6f37f7b7eed1b5e27346691901c11

It's now optional. It will be set to "." by default in the GUI. But even
if omitted will again get set to "." at the lower levels.

Comment 3 Mike Foley 2011-12-12 18:16:43 UTC
verified RHQ 3, master.  observed the default set to '.' ... and additionally tested with that '.' omitted.

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


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