tracker issue : CF-4126473

select a category, or use search below
(searches all categories and all time range)
Title:

[ANeff] ER for: action="redact" 'pages' attribute should be optional

| View in Tracker

Status/Resolution/Reason: Closed/Fixed/Fixed

Reporter/Name(from Bugbase): Aaron Neff / Aaron Neff (Aaron Neff)

Created: 03/09/2016

Components: Document Management

Versions: 2018

Failure Type: Unspecified

Found In Build/Fixed In Build: Beta2_v12 / 310147

Priority/Frequency: Minor / Unknown

Locale/System: English / Win All

Vote Count: 0

action="redact" 'pages' attribute should be optional

For action="merge", the pages attribute is optional and defaults to * (all pages)
For action="addstamp", the pages attribute is optional and defaults to * (all pages)
For action="redact", the pages attribute is required (it should default to * - all pages)

Repro:

<cfscript>
  cfhtmltopdf(destination=expandPath('./MyPDF1.pdf'), overwrite=true) {
	  writeOutput(repeatString("PDF 1 Page 1", 425));
	  cfhtmltopdfitem(type="pagebreak");
	  writeOutput(repeatString("PDF 1 Page 2", 425));
  };
  cfhtmltopdf(destination=expandPath('./MyPDF2.pdf'), overwrite=true) {
	  writeOutput(repeatString("PDF 2 Page 1", 425));
	  cfhtmltopdfitem(type="pagebreak");
	  writeOutput(repeatString("PDF 2 Page 2", 425));
  };
  cfpdf(action="merge", destination=expandPath('./MyPDFMerged.pdf'), overwrite=true) {
	  cfpdfparam(source=expandPath('./MyPDF1.pdf'));
	  cfpdfparam(source=expandPath('./MyPDF2.pdf'));
  };
  cfpdf(action="redact", source=expandPath('./MyPDFMerged.pdf'), destination=expandPath('./MyPDFRedacted.pdf'), overwrite=true) {
	  cfpdfparam(wordstoredact=["page"], ignorecase=true);//pages attribute should be optional and 1) default to all and 2) * means all (to be consistent w/ action="merge" and action="addstamp")
  };
  cfpdf(action="addstamp", source=expandPath('./MyPDFRedacted.pdf'), destination=expandPath('./MyPDFStamped.pdf'), overwrite=true) {
	  cfpdfparam(iconname="approved", coordinates="200,300,400,500");
  };
</cfscript>

Actual result:

-----------
Attribute validation error for CFPDFPARAM tag in cfscript.

It has an invalid attribute combination: ignorecase,wordstoredact. Possible combinations are:
Required attributes: 'coordinates,'. Optional attributes: 'content,iconname,ignorecase,pages,wordstoredact'.
Required attributes: None. Optional attributes: 'coordinates,ignorecase,pages,wordstoredact'.
Required attributes: 'source'. Optional attributes: 'description,encoding,filename,mimetype'.
Required attributes: None. Optional attributes: 'pages,password,ignorecase,source,wordstoredact'. 
-----------

Expected result: All 4 pages should be redacted

----------------------------- Additional Watson Details -----------------------------

Watson Bug ID:	4126473

External Customer Info:
External Company:  
External Customer Name: Aaron Neff
External Customer Email: adobelabs@itisdesign.com
External Test Config:

Attachments:

Comments:

Adding BUG AUDIT TRAIL ********action: updated fieldName: Priority newValue: 1 oldValue: 0 oprid: vmannebo recordName: RQ_DEFECT timpestamp: 2016-02-16 15:11:50.0 action: updated fieldName: Reason newValue: PRHaveInfo oldValue: PRNeedInfo oprid: prerelease recordName: RQ_DEFECT timpestamp: 2016-01-15 00:19:24.0 action: updated fieldName: Status newValue: ToTrack oldValue: ToTrack oprid: prerelease recordName: RQ_DEFECT timpestamp: 2016-01-15 00:19:24.0 action: updated fieldName: State newValue: Open oldValue: Open oprid: prerelease recordName: RQ_DEFECT timpestamp: 2016-01-15 00:19:24.0 action: updated fieldName: Reason newValue: PRNeedInfo oldValue: PRHaveInfo oprid: bihani recordName: RQ_DEFECT timpestamp: 2016-01-14 09:33:10.0 action: updated fieldName: Status newValue: ToTrack oldValue: ToTrack oprid: prerelease recordName: RQ_DEFECT timpestamp: 2016-01-12 09:33:44.0 action: updated fieldName: Reason newValue: PRHaveInfo oldValue: PRNeedInfo oprid: prerelease recordName: RQ_DEFECT timpestamp: 2016-01-12 09:33:44.0 action: updated fieldName: State newValue: Open oldValue: Open oprid: prerelease recordName: RQ_DEFECT timpestamp: 2016-01-12 09:33:44.0 action: updated fieldName: Reason newValue: PRNeedInfo oldValue: Blank oprid: bihani recordName: RQ_DEFECT timpestamp: 2016-01-12 08:50:27.0 action: updated fieldName: Status newValue: ToTrack oldValue: Unverified oprid: bihani recordName: RQ_DEFECT timpestamp: 2016-01-12 08:50:27.0
Comment by CFwatson U.
4035 | March 09, 2016 02:14:09 AM GMT
Added By: PreRelease User User Name:Aaron Neff Note Added: Hi Kailash, Regarding addstamp: Currently pages attribute defaults to *. That will remain the same/unchanged, right? Regarding redaction: By "next update" are you referring to Raijin Public Beta? It should be done before final release. Otherwise, after release, the team will say it cannot be changed due to backward-compat. Thanks!, -Aaron Date Added :2016-01-15 00:19:24.0 Added By:bihani Note Added: Hi Aaron, We will have discussion within the team and if required, we will provide the fix in the next update. We will not be taking up this fix at this point of time. Thanks, Kailash Date Added :2016-01-14 09:33:12.0 Added By: PreRelease User User Name:Aaron Neff Note Added: Can we please continue discussion here?: http://prerelease.adobe.com/r/?4e98fd9c4e1e419687f7892ee3c7859a I don't always get notified when engineers add comments on tickets. Thanks!, -Aaron Date Added :2016-01-12 09:34:39.0 Added By: PreRelease User User Name:Aaron Neff Note Added: Hi Kailash, Why is that helpful at all? The most helpful is if pages defaults to *. Thanks!, -Aaron Date Added :2016-01-12 09:33:42.0 Added By:bihani Note Added: Hi Aaron, We have evaluated this and we are making pages mandatory for both addStamp and Redaction. It should fix this issue as an exception will be thrown if you try to run the code without specifying any page number. Thanks, Kailash Date Added :2016-01-12 08:50:28.0 Added By: PreRelease User User Name:Aaron Neff Note Added: The forum thread discussing this ticket is: http://prerelease.adobe.com/r/?4e98fd9c4e1e419687f7892ee3c7859a Please follow-up in that thread, not here in this ticket. Date Added :2016-01-11 07:41:02.0 Added By: PreRelease User User Name:Aaron Neff Note Added: In the script comment, please ignore this part: "and 2) * means all" Date Added :2016-01-11 06:46:30.0 Added By: PreRelease User User Name:Aaron Neff Note Added: Entered Feature. Date Added :2016-01-11 06:39:49.0
Comment by CFwatson U.
4036 | March 09, 2016 02:14:11 AM GMT
Hi Aaron, Though it is not difficult to add pages as default for both stamps and redaction, we didn't do it as the general use case for these features might be where users want to add different stamps on different pages, or add stamps only to specific pages. Similarly for redaction, an end user will want to redact different parts of different pages, or redact different words for specific pages. So we thought instead of giving pages as optional, we will let the end user decide where he wants to redact/add stamp Thanks, Kailash
Comment by Kailash B.
4037 | July 22, 2016 02:59:22 AM GMT