Closed (fixed)
Project:
Image Assist
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Sep 2008 at 20:08 UTC
Updated:
25 Oct 2008 at 08:53 UTC
Jump to comment: Most recent file
Comments
Comment #1
jsm174 commentedI'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
Comment #2
zoo33 commentedThat's probably the reason, yes.
Comment #3
jsm174 commentedI 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
Comment #4
sunBased on the returned information in this hook, I guess we should move views include files into a sub-directory now?
Comment #5
jsm174 commentedI 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.
Comment #6
zoo33 commentedIt 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. :)
Comment #7
zoo33 commented@jsm174: If you're completely stuck, maybe you can post a new patch with what you did so far? Thanks!
Comment #8
sunYes, 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():
Comment #9
sunI've decided against keeping the old include files, since we supply the proper (new) path in all cases. So, Thanks, committed.
Comment #10
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #11
naheemsays commentedUsing the latest dev release from 15 Sept I still get the errors when combined with views rc-3:
on the modules page, and more in the img_assist popup window.
Comment #12
naheemsays commentedComment #13
atiras commentedAlso using latest dev release from 15 Sept, and Views rc-4, same error:
Comment #14
sunThis issue has been fixed already. See #308005: Theme registry needs template file in /includes for a patch to fix those file inclusion errors.
Comment #15
atiras commentedThanks. I'll go away and teach myself how to apply a patch...
Comment #16
kniekel commentedCould 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.
Comment #17
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.