og_user_roles doesn't work with OG 6.x-2.x-rc3

Josh Waihi - July 23, 2009 - 00:55
Project:OG User Roles
Version:6.x-1.5
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:reviewed & tested by the community
Description

queries are still trying to use og_ancestry.is_public but is doesn't exist anymore:

<?php
// Migrate the is_public column from og_ancestry to a new og_nid table. Denormalizing.
function og_update_6203() {
 
$ret = array();
  if (
module_exists('og_access')) {
   
// Do nothing. An og_access update handles this.
 
}
  else {
   
db_drop_field($ret, 'og_ancestry', 'is_public');
   
og_ancestry_dedupe($ret);
  }

  return
$ret;
}
?>

#1

Josh Waihi - July 23, 2009 - 01:46

here is a patch that replaces all the instances of is_public. Please review that I'm doing the right thing here, I'm not sure, but I think its in the right direction.

AttachmentSize
528154-replace-is_public.patch 2.72 KB

#2

Josh Waihi - July 23, 2009 - 04:39
Status:active» reviewed & tested by the community

patch is working for me.

#3

snorkers - August 6, 2009 - 16:57

Seems to be working for me too. Thanks for posting the patch

#4

tallsimon - August 23, 2009 - 14:18

any chance this could go into the dev version so I can fix the site without applying patch? foolishly applied upgrade to live site having not noticed this error on testing

#5

tallsimon - August 24, 2009 - 19:47

or could anyone post a zip file with the module with the patch applied? would be very grateful...

#6

snorkers - August 31, 2009 - 15:53

Patched module attached @tallsimon.

For info, there's a wealth of info in applying patches at http://drupal.org/patch/apply

AttachmentSize
og_user_roles.module.zip 37.25 KB

#7

tallsimon - September 1, 2009 - 11:05

thank you!

#8

GA - October 8, 2009 - 18:31

patch is working for me too.

#9

nemchenk - October 26, 2009 - 18:06

Patch works for me with og-6.x-2.0 and og_user_roles-6.x-1.5

 
 

Drupal is a registered trademark of Dries Buytaert.