Closed (fixed)
Project:
CiviRelate
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
2 Mar 2011 at 18:47 UTC
Updated:
27 Apr 2011 at 17:54 UTC
Jump to comment: Most recent file
Version 4.0 of CiviCRM will sport a new api interface and be compatible with Drupal 7 - we need to keep up!
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | update_code_for_d7_1079036-1.patch | 5.2 KB | lkacenja |
Comments
Comment #1
lkacenjaHi,
Your module did exactly what we needed for a client's site. Thanks for the work. Simple and elegant. Works well. We ported it to D7, but kept the Civi API at Version 2 as it works fine and is still available in Civi 4.x. Hope this helps. Thanks again.
Comment #2
fathershawnThanks for the patch, and I'm happy to have a look at it! I saw your patch just after I pushed a D7 dev branch to the project, so have a look at my approach while I review yours!
Comment #3
fathershawnWe both made the same changes. The only difference is that I moved the form api implementation code for admin/settings... into a separate file. The speed boost is minor since the form is so small but no need to parse it on every page load. I'll next move the api code to v.3 both to test the new v.3 implementation and for compatibility going forward. I've also improved the error traps to log the errors via watchdog.
Comment #4
lkacenjaHaha funny timing. I'll take a look at your release sometime soon, and consider applying it to the site we used. Thanks again for the module :)
Comment #5
fathershawnMigrated to CiviCRM v.3 api and everything seems to work.
Comment #6
fathershawn