Auto Generate Whitelist Data Install Script

[mpchadwick]

Rather than just doing this

supee-6788-toolbox/fixSUPEE6788.php

Line 705 in 186ecb2

$this->_write->insertMultiple( $this->_blocksTable, array_values( $inserts ) );

it might be nice if this created a data install script for the necessary blocks and variables (perhaps optionally). That way you can deploy the upgrade script rather than running this in production. Magento "technical details" also recommend a data install script. I'm sure everyone is busy getting this patch out, but if anyone has time (I can maybe, but probably not PR this myself).

[rhoerr]

The tech details recommendation was in regards to affected modules, and in that context it does make sense. Creating a module, you'd want to make sure any blocks needed are added to the whitelist when it gets installed.

To do that, though, you need a module to attach the data upgrade to. Don't have that in this context. You could generate everything needed, for sure, but that's getting rather involved.

[mpchadwick]

Yeah, it certainly wouldn't be trivial, but would be nice...even just to get the install script only would save some time and then it would be on the developer to create the module to make sure it gets executed...

[chelevich]

created a PR to get the upgrade scripts from log file