The text says the image module is not a dependency, but when running this on a new install that has never had the image module enabled, I get the following error on every page:

user warning: Table 'cdnmdev_7.image' doesn't exist query: SELECT * FROM image i INNER JOIN imagefield_assist_map iam ON i.nid = iam.iid WHERE i.nid = 17 in [PATH_TO_MODULE]/imagefield_assist/imagefield_assist.module on line 1461.

It seems to work even so, so I'm guessing this is some old code that is not needed if the image module is not enabled so it should be wrapped in a if (module_exists()) or something.

Comments

lourenzo’s picture

Status: Active » Needs work

Actually, it's old code that needs to be removed...

Working on that...

akaserer’s picture

in my case i still needed this table with the latest version - ImageField Assist 6.x-1.0-beta3

i checked all the .install files on cvs and didnt find the create table statement.

i created the table manuelly:

CREATE TABLE `imagefield_assist_map` (
`nid` int(10) unsigned NOT NULL default '0',
`iid` int(10) unsigned NOT NULL default '0',
PRIMARY KEY (`nid`),
KEY `fid` (`iid`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8;

not sure if "int(10) unsigned NOT NULL default '0'," is exactly correct, but it works.

lourenzo’s picture

Actually this dependency is not real, but instead a bug.
I'll be working on that soon.

Is this bug present on the beta release?

Thanks

akaserer’s picture

Version: 6.x-1.x-dev » 6.x-1.0-beta3
Issue tags: +imagefield_assist_map

Is this bug present on the beta release?

yes

i use: ImageField Assist 6.x-1.0-beta3

lourenzo’s picture

Status: Needs work » Fixed

Fixed, adding db_table_exists conditionals. Soon with the new maintenance cycle, all code not useful (including any image module integration) will be removed.
Released the fix since beta5.

lourenzo’s picture

Assigned: Unassigned » lourenzo
lourenzo’s picture

Status: Fixed » Closed (fixed)