Closed (fixed)
Project:
Sampler API
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
9 Dec 2010 at 18:43 UTC
Updated:
31 Aug 2012 at 20:11 UTC
Jump to comment: Most recent file
I would love to use the sampler_api - on a Drupal 7 site.
Any suggestions on a path to follow to do the upgrade ? (I will keep poking around, but having not looked through this before, there is a lot to get my head around.)
Yeah, I know Ctools is not stable yet, but are the elements of ctools that sampler requires in any kind of fit state ?
Thanks for any info.
Peter
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 993850-sampler-d7-port-4.interdiff.patch | 3.4 KB | hunmonk |
| #3 | 993850-sampler-d7-port-3.patch | 46.7 KB | elc |
Comments
Comment #1
hunmonk commentedi think a lot of the API will not need to be adjusted. pretty sure the big area that will need attention is the database handlers.
sampler api only uses the ctools plugin architecture -- no idea if that's ready or not yet.
Comment #2
gregglesSubscribing and fixing some metadata.
Comment #3
elc commentedI had a project that needed this in D7, but it has since been postponed and my focus has been directed elsewhere. I didn't get far, but this now slots itself into D7, but should in no way be considered a complete port. It was mostly just following Coder, and then changing the SQL where it had conditional changes on the SQL, and where methods no longer existed.
I don't think it actually works properly yet :P There's a whole DB abstraction layer in there that could probably be eliminated in favour of just using DB:TNG unless I'm way off the mark.
Anyway, hope this is a step in the right direction for someone. The patch from a branch off at 8cc7aa4e1233a5120d58269a65dc609bf6e55078.
Comment #4
hunmonk commentedapologies for the long radio silence. had time to review this today, it's a great start. the attached interdiff shows the few minor changes i made to your initial patch:
As there is actually a check for taxonomy module in that function.
i've created a 7.x-1.x branch and release node, and pushed the changes there, so we are underway!
hoping to make good progress in the next day or two.
Comment #5
hunmonk commentedok, the 7.x conversion is now mostly done. still need to test everything.
i also looked at the idea of tossing the storage plugins in favor of the new storage API, but i don't think that's really feasible, as the storage plugins do more than just abstract the database engine away.
will try to knock out most of the testing tomorrow.
Comment #6
hunmonk commentedthe upgrade is basically complete. i've tested all of the drush functionality, storage with all of the various storage plugins, the menu callbacks that perform sampling, and all of the metric plugins appear to be working.
the only thing that hasn't been looked at is the views integration, which i'm not very familiar with, and could use some support on.
leaving the issue open until that's addressed.
Comment #7
jhodgdonFound one minor issue with the views integration:
#1706716: PHP warning after D7 cache clear
Comment #8
hunmonk commentedhad a chance to test the views integration on a 7.x site today, and it seems to work perfectly. so, i consider the initial 7.x port done, and have created a 7.x-1.0 release.