Requested by add1sun.

CommentFileSizeAuthor
#5 demo_d7.patch886 bytescwgordon7
demo-HEAD.patch11.56 KBsun

Comments

add1sun’s picture

Thank 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.

add1sun’s picture

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!!

sun’s picture

Status: Needs review » Needs work
sun’s picture

Status: Needs work » Active

Committed the existing patch as is for now. Hopefully, someone can fix the remaining bug(s). Hence, reverting to active.

cwgordon7’s picture

Status: Active » Needs review
StatusFileSize
new886 bytes

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.

sun’s picture

Status: Needs review » Fixed

Thanks 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.

Status: Fixed » Closed (fixed)

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