Project

General

Profile

Actions

Bug #2465

closed

PT/technique "checkGenericFileContent" has a mandatory field to enforce file content

Added by Jonathan CLARKE almost 12 years ago. Updated almost 12 years ago.

Status:
Released
Priority:
2
Assignee:
Matthieu CERDA
Category:
Techniques
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

This PT can now do several things: enforce a file content, delete lines, replace lines... But if you choose to just replace lines, you still have to specify some content to put into the file, which makes no sense.

1. This field should be made optional.

2. Also, the PT's layout should be clarified, to make it more obvious that you must choose a file name, then some actions, then configure these actions. I propose the attached patch for the policy.xml file but it needs checking with regard to reporting and components.

3. The variable "Enforce the content of the file:" should be a boolean, the same as the following 2, not a SELECT1 with 2 values.


Files


Related issues 1 (0 open1 closed)

Has duplicate Rudder - Bug #2208: PT Enforce a file content: Deleting some lines in a file require to specify a content into the file.Rejected2012-01-27Actions
Actions #1

Updated by Matthieu CERDA almost 12 years ago

  • Status changed from New to Pending technical review
  • % Done changed from 0 to 100

Applied in changeset commit:c74bca73863c250c459210b4b300aadb66774190.

Actions #2

Updated by Matthieu CERDA almost 12 years ago

I applied your patch, which is valid, but without the 'component="true"' part that would break reporting.

Actions #3

Updated by Jonathan CLARKE almost 12 years ago

  • Status changed from Pending technical review to 2
  • % Done changed from 100 to 50

I'm really not sure about this. There is now a component named "File content" which never has any reports:

 <SECTION name="File content" component="true" componentKey="GENERIC_FILE_CONTENT_PATH">

I have no doubts as to whether this will break reporting. Did you test this? Please review.

Actions #4

Updated by Jonathan CLARKE almost 12 years ago

  • Target version changed from 2.3.8 to 2.3.7
Actions #5

Updated by Matthieu CERDA almost 12 years ago

  • Status changed from 2 to Pending technical review
  • % Done changed from 50 to 100

Applied in changeset commit:a8307ad80b1e2cb1eea200c2bf477a627e8447a7.

Actions #6

Updated by Matthieu CERDA almost 12 years ago

You were right, this " <SECTION name="File content" component="true" componentKey="GENERIC_FILE_CONTENT_PATH">" part should not be here... I made it component="false", now we get all the expected reports.

Actions #7

Updated by Nicolas PERRON almost 12 years ago

Tested and approved ! It's OK for me. Thanks Matthieu

Actions #8

Updated by Jonathan CLARKE almost 12 years ago

  • Status changed from Pending technical review to Released

Awesome, thanks for testing guys.

Actions

Also available in: Atom PDF