Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#49029 closed enhancement (fixed)

Introduce a JSON file into the personal data export.

Reported by: xkon Owned by: xkon
Priority: normal Milestone: 5.4
Component: Privacy Version:
Severity: normal Keywords: has-patch commit has-unit-tests has-dev-note
Cc: Focuses:

Description

There have been discussions for supporting a JSON format as well for a long time now (including the cross-privacy group) but it was never implemented or was skipped for other priorities.

I think it would be a +1 if we could introduce this in 5.4.

The patch creates an "export.json" that will be compressed in the produced zip along with the current index.html.

I've reorganized the code on the exporter a bit as well so we can benefit from 1 data group going into 2 different files as needed.

This was also mentioned as an extra request in #46424.

Attachments (4)

49029.diff​ (5.0 KB ) - added by xkon 7 years ago.
49029.2.diff​ (5.5 KB ) - added by xkon 7 years ago.
49029.3.diff​ (7.6 KB ) - added by xkon 7 years ago.
49029.4.diff​ (1.0 KB ) - added by xkon 7 years ago.

Download all attachments as: .zip

Change History (13)

@xkon
7 years ago

#1 @xkon
7 years ago

  • Owner set to xkon

@xkon
7 years ago

#2 @xkon
7 years ago

I missed adding the $json_report_pathname to the wp_privacy_personal_data_export_file_created action on the previous patch.

49029.2.diff​ appends the extra pathname at the end of the action to avoid breaking any possible existing calls.

@xkon
7 years ago

#3 @xkon
7 years ago

  • Keywords commit has-unit-tests added; 2nd-opinion removed

49029.3.diff​ adds phpunit tests as well for the JSON file.

I've tested again all default exports, as well as content from extra plugins and everything, seems to be working as expected.

Marking this for commit.

#4 @audrasjb
7 years ago

  • Keywords needs-dev-note added

#5 @SergeyBiryukov
7 years ago

  • Resolution → fixed
  • Status assigned → closed

In 47146:

Privacy: Introduce a JSON file into the personal data export.

The produced ZIP archive will now include an export.json file along with the current index.html.

Props xkon.
Fixes #49029. See #46424.

@xkon
7 years ago

#6 @xkon
7 years ago

  • Resolution fixed
  • Status closed → reopened

Sorry @SergeyBiryukov ! Reopening this for a missed doc on the created file action.

49029.4.diff​ takes care of that also 😅.

#7 @SergeyBiryukov
7 years ago

  • Resolution → fixed
  • Status reopened → closed

In 47150:

Docs: Document the addition of $json_report_pathname parameter to the wp_privacy_personal_data_export_file_created action.

Follow-up to [47146].

Props xkon.
Fixes #49029.

#8 @garrett-eclipse
7 years ago

  • Keywords has-dev-note added; needs-dev-note removed

#9 @garrett-eclipse
6 years ago

  • Focuses privacy removed

Dropping privacy focus as it's already in the Privacy component.

Note: See TracTickets for help on using tickets.