The Views integration probably needs an update, according to this message on the dev list:

a new hook must be implemented by all Views modules: hook_views_api(). Without the proper use of this hook, the *.views.inc files will not be included.

I'll try to find some time to look into it.

Comments

jsm174’s picture

I'm guessing this might be related:

I just upgraded to the latest dev versions of views and cck. When the image assist popup is rendered, I get:

Fatal error: Call to a member function set_arguments() on a non-object in C:\xampplite\htdocs\drupal\sites\all\modules\img_assist\img_assist.module on line 649

__________________________________________

Error: The specified view was not found.

-- Jason

zoo33’s picture

Category: task » bug

That's probably the reason, yes.

jsm174’s picture

StatusFileSize
new518 bytes

I followed the first part of KarenS's instructions at:

http://www.angrydonuts.com/attention-views-api-developers#comment-2963

Then I emptied my cache and img_assist was back in action.

I know it's a super simple change, but attached is the patch.

-- Jason

sun’s picture

Status: Active » Needs work

Based on the returned information in this hook, I guess we should move views include files into a sub-directory now?

jsm174’s picture

I went to re-do the patch to help out. I moved the *.token.inc file to /includes and both *views*.inc files into /includes/views (like CCK does). I thought I had all the path references adjusted, but when I empty the cache, I get:

* warning: include_once(sites/all/modules/img_assist/img_assist.views.inc) [function.include-once]: failed to open stream: No such file or directory in D:\HostingSpaces\site.com\wwwroot\includes\theme.inc on line 286.

* warning: include_once() [function.include]: Failed opening 'sites/all/modules/img_assist/img_assist.views.inc' for inclusion (include_path='.;C:\php5\pear') in D:\HostingSpaces\site.com\wwwroot\includes\theme.inc on line 286.

I'm not super familiar with the D6 theming system yet, and nothing is jumping out at me.

zoo33’s picture

Assigned: zoo33 » Unassigned

It would be very nice if we could get this working. You would actually be able to find what you're looking for in the img_assist directory. :)

zoo33’s picture

@jsm174: If you're completely stuck, maybe you can post a new patch with what you did so far? Thanks!

sun’s picture

Status: Needs work » Needs review
StatusFileSize
new17.28 KB

Yes, moving files around is not that easy... since we also need to ensure that old files are no longer existent.

Also needed to add the following line to img_assist_views_default_views():

$view->get_total_rows = TRUE;
sun’s picture

Status: Needs review » Fixed

I've decided against keeping the old include files, since we supply the proper (new) path in all cases. So, Thanks, committed.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

naheemsays’s picture

Using the latest dev release from 15 Sept I still get the errors when combined with views rc-3:

* warning: include_once(sites/all/modules/img_assist/img_assist.views.inc) [function.include-once]: failed to open stream: No such file or directory in /var/www/vhosts/theraggedyedge.co.uk/httpdocs/includes/theme.inc on line 286.
* warning: include_once() [function.include]: Failed opening 'sites/all/modules/img_assist/img_assist.views.inc' for inclusion (include_path='.:') in /var/www/vhosts/theraggedyedge.co.uk/httpdocs/includes/theme.inc on line 286.

on the modules page, and more in the img_assist popup window.

naheemsays’s picture

Status: Closed (fixed) » Active
atiras’s picture

Also using latest dev release from 15 Sept, and Views rc-4, same error:

* warning: include_once(sites/all/modules/img_assist/img_assist.views.inc) [function.include-once]: failed to open stream: No such file or directory in C:\xampp\htdocs\drupal\includes\theme.inc on line 286.
* warning: include_once() [function.include]: Failed opening 'sites/all/modules/img_assist/img_assist.views.inc' for inclusion (include_path='.;C:\xampp\php\pear\') in C:\xampp\htdocs\drupal\includes\theme.inc on line 286.

sun’s picture

Status: Active » Fixed

This issue has been fixed already. See #308005: Theme registry needs template file in /includes for a patch to fix those file inclusion errors.

atiras’s picture

Thanks. I'll go away and teach myself how to apply a patch...

kniekel’s picture

Could anyone who is able to do it just apply the patch and release a new developer version of img_assist? That would be nice for those like me who are just too dumb to apply it themselves. Thank you.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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