Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bricel’s picture

FileSize
7.96 KB
amitaibu’s picture

Status: Active » Needs work
+++ b/includes/og_vocab.services.incundefined
@@ -0,0 +1,163 @@
+              'description' => 'The id of the og vocab to get',

id => ID
og vocab => OG-vocab
End with dot.

+++ b/includes/og_vocab.services.incundefined
@@ -0,0 +1,163 @@
+ * @return bool

No need to define "bool".

+++ b/includes/og_vocab.services.incundefined
@@ -0,0 +1,163 @@
+  return user_access('og vacab services');

typo.

+++ b/includes/og_vocab.services.incundefined
@@ -0,0 +1,163 @@
+ * @return

Missing line space.

+++ b/includes/og_vocab.services.incundefined
@@ -0,0 +1,163 @@
+  // Adds backwards compatability with regression fixed in #1083242

Add full link.

+++ b/og_vocab.moduleundefined
@@ -606,6 +622,19 @@ function og_vocab_relation_get_by_group($group_type, $gid) {
+function og_vocab_get_by_taxonomy_vocabulary($vid) {

Missing params

amitaibu’s picture

+++ b/includes/og_vocab.services.incundefined
@@ -0,0 +1,163 @@
+    'og vacab services' => array(

Typo.

+++ b/includes/og_vocab.services.incundefined
@@ -0,0 +1,163 @@
+      'description' => t('Enable to use '),

Missing sentence?

bricel’s picture

See fixed patch.

bricel’s picture

Status: Needs work » Needs review
bricel’s picture

Moved uuid sync to hook_enable.

bricel’s picture

Minor fix, clearing cache before sync.

bricel’s picture

Uppps... forgot to commit change before saving patch.
New patch attahced.

bricel’s picture

Fixed install issue.

bricel’s picture

Typos fix.