#49029 closed enhancement (fixed)
Introduce a JSON file into the personal data export.
Reported by: | xkon | Owned by: | xkon |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Privacy | Keywords: | has-patch commit has-unit-tests has-dev-note |
Focuses: | Cc: |
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)
Change History (13)
#3
@
5 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.
#6
@
5 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Sorry @SergeyBiryukov ! Reopening this for a missed doc on the created file action.
49029.4.diff takes care of that also 😅.
I missed adding the
$json_report_pathname
to thewp_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.