Closed (fixed)
Project:
CKEditor Link OG
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Reporter:
Created:
13 May 2012 at 14:05 UTC
Updated:
31 Jan 2015 at 14:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
btopro commentedmost likely I didn't change the function og_get_group_context to og_context. Try that. Working on stabilizing d7 across my code base in the near future
Comment #2
pomliane commentedUnfortunately, it did not solve the issue: same 500 error.
Comment #3
btopro commentedHmm.. can you try it with og 1.x? Not sure which to support yet as i'm new to OG in D7 and 2.x seems a bit new / unstable still
Comment #4
btopro commentedpushed some changes to dev, please check them out when it publishes tonight and let me know if it helped.
Comment #5
pomliane commentedComment #6
pomliane commentedTanks a lot for your super fast answer! :)
In og-7.x-2.x:
- table "og_membership" seems to be the new "og_ancestor".
- "gid" replaces "group_nid"
- "og.nid" should probably be "og.etid"
Once these elements modified, the error message gets back to:
I haven't tried your module on an og -7.x-1.x install yet, but, this issue looks like a feature request for og-7.x-2.x compatibility, doesn't it?
Comment #7
btopro commentedits probably an issue with both versions; 1.x and 2.x of 7 have schemas that are different from 6.x version so I'm sure that's the real issue. Will need to look into this at a later time once I play with OG in D7 (either version). A bit annoyed at how much tinkering has gone on with such a critical infrastructure project but oh well, if your able to generate more errors / find anything let me know. Definitely will support this module in D7, just need to get up to speed myself.
Comment #8
chris burge commentedAttached is patch for Drupal 7.x that supports OG 2.x.
Comment #9
btopro commentedThanks for the patch. Looks reasonable but I'm going to have mmilutinovic1313 test it
Comment #10
mmilutinovic1313 commentedI have applied Chris Burge's patch. It looks to be working and did not cause anything to break. Patch should be visable shortly. Message me if anything seems like it's not working or the patch doesn't apply.
Thanks!
Mark Milutinovic
Comment #11
mmilutinovic1313 commentedComment #12
btopro commentedmark - next time when you commit a patch set it to fixed and put the node id like this in the commit message -- #1576212
that way the commit shows up in this thread. Other then that awesome and this should be resolved, as well as another issues: http://cgit.drupalcode.org/ckeditor_link_og/commit/?id=f85c3af