Notice: in views_timelinejs_plugin_style_timelinejs->render() (line 232 of /XXX/modules/contrib/views_timelinejs/views_timelinejs_plugin_style_timelinejs.inc).

Offending code:

          if (isset($this->options['timeline_config']['link_text_enabled'])) {
            $rows[$count][$target_field] .= theme('views_timelinejs_link_to_entity', array('uri' => $row->uri['path'], 'link_text' => t($link_text)));

A couple things here; the root problem is that the row did not have a URI; this speaks to the larger issue of requiring the media to be an image field, which is kind of odd as Timeline JS just takes a bare URL.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

FluxSauce’s picture

Status: Active » Needs review
FileSize
963 bytes
FluxSauce’s picture

EDIT - ignore patch, re-rolling.

operinko’s picture

Status: Needs review » Fixed

The latest patch does not apply after applying the one from http://drupal.org/node/1855560.
As far as I can see, the only difference between this patch and that other patch is
&& isset($row->uri['path'])

I added that one, just to be on the safe side.

FluxSauce’s picture

I added that one, just to be on the safe side.

Cool, that sorts the issue. As I was talking through #1860744: Optional Link text always displayed with imclean I gathered that, but wasn't in a position to say duplicate so I just gave him good advice :-) The && isset($row->uri['path']) addresses the notice.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.