Drupal - How can I make an array globally accessible?

Question

I'm using this code in a views field template (in this case views-view-field--all-members--uid.tpl.php):

<?php

$users_friends = flag_friend_get_friends($user->uid);

$users_friends_ids = array();

foreach ($users_friends as $id => $value) {     
 $users_friends_ids[] = $id;    
}


?>

It basically gets the user ids of friends and puts them in an array so I can check if the field matches any of the user ids.

So my problem is that I don't want to have this within this template (for a few reasons), but if I don't I can't access the array. How can I make this array globally accessible?

Coder1 · Accepted Answer · 2011-04-18T20:34:41.823

5

Without knowing your "few reasons", I can't say if this is the answer for sure. My own reasons would probably be that I don't want the same code executing a bunch of times, and I'd rather not have the same exact code in multiple places.

I would then create a function with a static variable to hold the friends array.

function mymodule_get_friends_ids() {
  // pull in the current global user variable
  global $user;

  // call up the static variable
  static $users_friends_ids;
  // return if this static var has already been set
  if (is_array($users_friends_ids)) {
    return $users_friends_ids;
  }

  // if we hit here, then this function has not been
  // run yet for this page load.

  // init array
  $users_friends_ids = array();

  // if user is anon, no need to go on
  if (user_is_anonymous()) {
    return $users_friends_ids;
  }

  // get friends array
  $users_friends = flag_friend_get_friends($user->uid);

  // build ids array
  foreach ($users_friends as $id => $value) {     
    $users_friends_ids[] = $id;    
  }

  return $users_friends_ids;
}

Now in your templates, you can call mymodule_get_friends_ids() in as many places as you want, and the working code below the first return will only get executed the first time it is called.

edited Apr 18 '11 at 20:34

answered Apr 18 '11 at 09:23

Coder1

13,139
15
59
89

I saw another of your posts on how you are using this. There's actually no need to re-create that array of user ids. You could just use this function to set $users_friends (instead of $users_friends_ids) and then you can check to see if a specific user ID exists in that array with this: key_exists($some_uid, $users_friends) – Coder1 Apr 18 '11 at 09:39
How do you call the the module, is it by using module_invoke? Really struggling here as you can see :( – james6848 Apr 18 '11 at 10:22
I usually have a module for generic site helper functionality, but if you're not comfortable working with modules yet, you can just add that function to your template.php file. You should probably change the 'mymodule' in the function name to your theme name. Then, in every template you need to access that user ids array, just do something like $users_friends_ids = mymodule_get_friend_ids(); – Coder1 Apr 18 '11 at 10:26
I just can't seem to get this to work within a function. I've run some simple tests to see if the function is being being picked up and it is, plus the code works fine if taken from out of it's function wrapper and placed within a template. I'll keep plugging away. (I will mark you answer as 'accepted' once I've worked whats going on!) – james6848 Apr 18 '11 at 11:41
OK, there's something about returning arrays that I'm not grokking. Need to ask another question on S.O. I think... – james6848 Apr 18 '11 at 13:03
I added "global $user" to my answer. This may have been the problem. $user is a global variable not in scope of the function, so you need to define it accordingly. I was wrapped up in showing how to re-use a variable across the site that I forgot this. – Coder1 Apr 18 '11 at 16:11
1

I edited to add a return of the variable at the end. Coder1 did what I do a lot: forgetting to return the variable at the end of a function. You need to add the line after // ADDED LINE ... – James Apr 18 '11 at 20:23

score 2 · Answer 2 · answered Apr 18 '11 at 20:19

Coder1's advice is very good - it keeps you from populating your global variable namespace with a lot of junk. It's probably the most "elegant." It might not be the easiest to use if you are rather new to PHP (which I'm guessing might be the case if it's hard to get your head around returning arrays, but that's ok).

However, if this is really a priority, you probably don't care about having one extra global variable.

I suppose I may be stating the obvious here - but you can, at pretty much any point in execution (provided the information you need has already been generated - e.g., the $user variable has been populated), do this:

$GLOBALS['users_friends_ids'] = /* your code goes here */

Then in your template, you access this by ...

$friendsArray = $GLOBALS['users_friends_ids'];

Or you can simply use the construct

global $user_friends_ids;

when you want to initialize the variable, or access it inside a function or class (which is the case for your template files - they are called inside functions, so you need to globalize or use the $GLOBALS array, which is "automagically" all of the variables active in the global namespace).

The most "logical" place to do this would be inside a module using one of the many hooks available, to execute this code only once. hook_init() might do it for you, if the user object is already loaded at this point (not sure, you'll have to test). But you might not want to figure out making Drupal modules (it's not that difficult).

If you are doing this inside a template (and though it's not good practice, many Drupal site owners with a beginning knowledge of PHP put everything in templates), you'll want to know which template code is being executed when. Node template code tends to be executed before page template code - which is logical, since otherwise the variables for node content in the page template wouldn't be populated.

If you have listings of nodes, they'll be calling this code multiple times, so you'll end up doing something similar to what Coder1 is describing. If you don't want to create your own small module, you could put the function declaration he's written in your theme's template.php file, since it's called only once. You don't want to put function declarations in the tpl.php files, since they are sometimes called more than once (and you aren't allowed to declare functions more than once).

If you have a hard time understanding the function and the return, you can always do something like this in your code (which is very, very inelegant - but it's better to have inelegant code that you do understand, than elegant code that's you don't).

if(!isset($GLOBALS['users_friends_ids'])) {
   $GLOBALS['users_friends_ids'] = /* your code here */
}

NOTE: the above may be unnecessary, I saw that Coder1's elegant solution was simply missing a return variable at the end - and have edited it - the edit should appear "after peer review" — James, Apr 18 '11 at 20:25
Cheers. It may not be necessary but it's always good to have a bit of extra knowledge to get my teeth into. — james6848, Apr 19 '11 at 10:10

Drupal - How can I make an array globally accessible?

2 Answers2