Posted by sun on January 20, 2009 at 1:21am
| Project: | Demonstration site (Sandbox / Snapshot) |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Requested by add1sun.
| Attachment | Size |
|---|---|
| demo-HEAD.patch | 11.56 KB |
Comments
#1
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.
#2
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!!
#3
#4
Committed the existing patch as is for now. Hopefully, someone can fix the remaining bug(s). Hence, reverting to active.
#5
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.
#6
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.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.