Set Context User Target to Node Author

Yeah, I thought about making this into its own context when I was writing the default contexts. That's what you'll have to do -- you'll need to write a new module that adds this as a new context. Here's your .module file (sans closing ?>):

/**
 * @file
 *   Provides a node author context to the Facebook-style Statuses module.
 */

/**
 * Implementation of hook_facebook_status_context_info().
 */
function EXAMPLE_facebook_status_context_info() {
  $path = drupal_get_path('module', 'facebook_status');
  return array(
    'node_author' => array(
      'title' => t('Node author'),
      'description' => t('The stream belongs to the author currently viewed node, if applicable.'),
      'handler' => 'facebook_status_node_author_context',
      'view' => 'facebook_status_stream',
      'weight' => -2,
      'file' => $path .'/includes/utility/facebook_status.contexts.inc',
    ),
  );
}

/**
 * The node author context.
 */
class facebook_status_node_author_context extends facebook_status_user_context {
  function type() {
    return 'node_author';
  }
  function is_applicable() {
    return arg(0) == 'node' && is_numeric(arg(1));
  }
  function find_recipient() {
    return $this->is_applicable() ? $this->load_recipient(menu_get_object('node')->uid) : new stdClass();
  }
  function recipient_url($recipient) {
    return 'user/'. $this->recipient_id($recipient);
  }
}

If you need to use the status update form on other pages with other node types and you don't want this context to apply on them as well, you can either add a check to the is_applicable() method of the facebook_status_node_author_context class or you can edit the context's settings and change where it applies that way.

I'll think about making this available as a default context. For me to do that I would want a way to have contexts "disabled" by default so I'd need to add that feature first but I think it would be a good idea.

Oh, sorry. Yeah, unfortunately you need to add that line in the main body of the file to include FBSS' context file. You may also want to put the class in its own file (with the include of facebook_status.contexts.inc at the top).

EXAMPLE.module:

/**
* @file
*   Provides a node author context to the Facebook-style Statuses module.
*/

/**
* Implementation of hook_facebook_status_context_info().
*/
function EXAMPLE_facebook_status_context_info() {
  return array(
    'node_author' => array(
      'title' => t('Node author'),
      'description' => t('The stream belongs to the author currently viewed node, if applicable.'),
      'handler' => 'facebook_status_node_author_context',
      'view' => 'facebook_status_stream',
      'weight' => -2,
      'file' => drupal_get_path('module', 'EXAMPLE') .'/EXAMPLE.contexts.inc',
    ),
  );
}

EXAMPLE.contexts.inc:

/**
 * @file
 *   Provides Facebook-style Statuses contexts for the EXAMPLE module.
 */

module_load_include('inc', 'facebook_status', 'includes/utility/facebook_status.contexts.inc');

/**
* The node author context.
*/
class facebook_status_node_author_context extends facebook_status_user_context {
  function type() {
    return 'node_author';
  }
  function is_applicable() {
    return arg(0) == 'node' && is_numeric(arg(1));
  }
  function find_recipient() {
    return $this->is_applicable() ? $this->load_recipient(menu_get_object('node')->uid) : new stdClass();
  }
  function recipient_url($recipient) {
    return 'user/'. $this->recipient_id($recipient);
  }
}

It's a class, not a function: http://php.net/oop

And that *is* the reason for the error but I don't know why that class wouldn't be available if the correct file was included. Try using require_once instead of module_load_include().

Just having the context in your system might not be enough -- you probably have to go to admin/settings/facebook_status/context and make sure that the new context takes priority over the User Profiles and Nodes contexts.

Cool. Glad it works.

I'm going to hold on to this then... will consider committing it.

Postponed pending #1261720: Allow disabling contexts

I had the below all typed up when I discovered I forgot the change the path for my include path:

'file' => drupal_get_path('module', 'EXAMPLE') .'/fourh_fbss.contexts.inc',

But thought I'd post this anyway just in case somebody else had the problem.

---

When I try to save the configuration page and give the node author priority, I get the white screen of death with this error:

PHP Fatal error:  Class 'facebook_status_node_author_context' not found in sites/all/modules/facebook_status/facebook_status.module on line 620, referer: admin/settings/facebook_status/contexts

I went into the facebook_status_contexts table and manually changed the weights so that node_author is -3 and user is -4. Then the fatal error when away, but now I'm back to square one-- the context is never hit, user is taking priority.

I thought this might be a module weight issue-- but tried 100 and -100 with no luck.

I got this working, but since I'm using panels, Drupal does know about a path that isn't node/[nid] -- it's member/[name]. Here's the context include file code:

/**
 * @file
 *   Provides Facebook-style Statuses contexts for the EXAMPLE module.
 */

module_load_include('inc', 'facebook_status', 'includes/utility/facebook_status.contexts');

/**
* The node author context.
*/
class facebook_status_node_author_context extends facebook_status_user_context {
  function type() {
    return 'node_author';
  }
  function is_applicable() {
    return arg(0) == 'member';
  }
  function find_recipient() {
		$account = user_load(array('name' => check_plain(arg(1))));
    return $this->is_applicable() ? $this->load_recipient($account->uid) : new stdClass();
  }
  function recipient_url($recipient) {
    return 'user/'. $this->recipient_id($recipient);
  }
} 

$account = user_load(array('name' => check_plain(arg(1))));

check_plain() and other filtering functions are only for use on output (i.e. when displaying information). Using it on input can sometimes cause incorrect results.

Thanks for the tip.

And I was wrong about needing to change the code-- your example syntax is fine. Long story. :-)

Please disregard the previous version of this comment that reported that the context configuration pages were always showing node author. I discovered I had some errant debugging code that was causing the problem. Sorry if you spent any time on this.

Now that #1261720: Allow disabling contexts has landed we can finally look into getting this committed.

Untested patch attached. (Make sure to enable the new context before testing.)

One thing I forgot to put into this patch is all the places Statuses checks for type = 'user' where it should now check for type IN ('user', 'node_author') (or $type == 'user' || $type == 'node_author'). While this mostly won't affect the user experience, we need to go through the many places this check exists and see if it needs changing (hook_user_delete() is one example). I'm okay with addressing this as a follow-up though.

Per #22, attached is a list of everywhere that might need changing.