Currently sanitizing works just by sanitizing the dump, but in some use cases the database may be copied in a different way. E.g. sql-sync.

CommentFileSizeAuthor
#4 sanitize_environment-2958104-3.patch2.82 KBbarthje
#2 sanitize_environment-2958104-2.patch2.18 KBbarthje

Comments

barthje created an issue. See original summary.

barthje’s picture

barthje commented
StatusFileSize
new sanitize_environment-2958104-2.patch2.18 KB

I've created a patch for the functionality.

I'd rather create an abstract class with all the base functionality and let the SqlDump qnd SqlSanitize classes extend from there, but not sure if you guys agree.

barthje’s picture

barthje commented
Status: Active » Needs review
barthje’s picture

barthje commented
StatusFileSize
new sanitize_environment-2958104-3.patch2.82 KB

Forgot to include the new service in my patch.

mycw1991’s picture

mycw1991 commented
Status: Needs review » Reviewed & tested by the community

Code looks good and did a functional test on drupal 8.5.1. Works as expected.

  • mhavelant committed eaeffce on 8.x-1.x authored by barthje 
    Issue #2958104 by barthje, mycw1991: Add the option to sanitize the...
mhavelant’s picture

mhavelant commented
Status: Reviewed & tested by the community » Fixed

@barthje
Thanks a lot!

Status: Fixed » Closed (fixed)

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