Wheee, it totally works. I didn't do any crazy stuff to really push it but it works as advertised. Made a few snapshots, set up cron and it all went smoothly. I did get a bunch of notices when creating a new snapshot:
* Notice: Undefined property: stdClass::$native_type in _demo_get_fields() (line 181 of /Users/addi/Sites/helpdemo/sites/all/modules/demo/database_mysql_dump.inc).
* Notice: Undefined property: stdClass::$native_type in _demo_get_fields() (line 182 of /Users/addi/Sites/helpdemo/sites/all/modules/demo/database_mysql_dump.inc).
* Notice: Undefined property: stdClass::$native_type in _demo_get_fields() (line 184 of /Users/addi/Sites/helpdemo/sites/all/modules/demo/database_mysql_dump.inc).
but all-in-all it works quite well enough to set up some D7 demos. Thank you!!
The attached patch was necessary to get the reset functionality to work for the 7.x branch, I can create a new issue if you'd like, but I thought this was a pretty crucial part of the port to 7.x.
Comments
Comment #1
add1sun commentedThank you! I haven't had time to test yet but I have a long flight tomorrow so I'll have several hours to play with this.
Comment #2
add1sun commentedWheee, it totally works. I didn't do any crazy stuff to really push it but it works as advertised. Made a few snapshots, set up cron and it all went smoothly. I did get a bunch of notices when creating a new snapshot:
but all-in-all it works quite well enough to set up some D7 demos. Thank you!!
Comment #3
sunComment #4
sunCommitted the existing patch as is for now. Hopefully, someone can fix the remaining bug(s). Hence, reverting to active.
Comment #5
cwgordon7 commentedThe attached patch was necessary to get the reset functionality to work for the 7.x branch, I can create a new issue if you'd like, but I thought this was a pretty crucial part of the port to 7.x.
Comment #6
sunThanks for reporting, reviewing, and testing! Committed to HEAD.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.