The profile 2 module also contains user data. The serializers should support `EntityValueWrapper`

CommentFileSizeAuthor
#5 gdpr_export-2959945-5.patch2.59 KBaczietlow
#2 gdpr_export-2959945-2.patch1.82 KBaczietlow
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aczietlow created an issue. See original summary.

aczietlow’s picture

aczietlow CreditAttribution: aczietlow at Mindgrub Technologies commented
Status: Active » Needs review
FileSize
gdpr_export-2959945-2.patch1.82 KB

Added profile2 bundles to the export

sepgil’s picture

sepgil CreditAttribution: sepgil at jobiqo - job board technology commented
Status: Needs review » Needs work

Yeah, makes sense to add a default profile normalizer. Thanks for the patch, but it seems the actual code for the normalizer is somehow missing:

diff --git a/include/profile2_normalizer.inc b/include/profile2_normalizer.inc
deleted file mode 100644
index e69de29..0000000

And you created the patch from the wrong git repo:

diff --git a/modules/gdpr_export_profile2/gdpr_export_profile2.info b/modules/gdpr_export_profile2/gdpr_export_profile2.info
aczietlow’s picture

aczietlow CreditAttribution: aczietlow at Mindgrub Technologies commented

There's no facepalm status to update this to for that....

aczietlow’s picture

aczietlow CreditAttribution: aczietlow at Mindgrub Technologies commented
Status: Needs work » Needs review
FileSize
gdpr_export-2959945-5.patch2.59 KB

Rerolling patch to include fixes this time.

  • sepgil committed 678474e on 7.x-1.x authored by aczietlow 
    Issue #2959945 by aczietlow, sepgil: Support for Profile2
sepgil’s picture

sepgil CreditAttribution: sepgil at jobiqo - job board technology commented
Status: Needs review » Fixed

Looks good, I adapted the readme and commited the patch. I wouldn't worry about passing the entity name staticly. Afterall it's just a string in an hook_entity_info()...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.