Upload module is not available for Drupal 7 and so Inline for D7 really shouldn't require it.

I'm not (yet) adept enough at PHP or Drupal development to really investigate the code in the module to see if it just a matter of dependencies[] = upload being in the .info file or if there is much more to it in the .module file.

CommentFileSizeAuthor
#1 inline-fix-dependency-1173396-1.patch511 bytesbyrond

Comments

byrond’s picture

byrond commented
Status: Active » Needs review
StatusFileSize
new inline-fix-dependency-1173396-1.patch511 bytes

In Drupal 7, the core File module provides similar service as Upload did in D6.

http://drupal.org/documentation/modules/file

Changing dependencies[] from 'upload' to 'file' seems to be sufficient.

drzraf’s picture

drzraf commented

status: => needs commit

ao2’s picture

ao2 commented

@drzraf if you actually tested the patch and it works for you, you can change the status to RTBC.

Regards,
Antonio

drzraf’s picture

drzraf commented

as I didn't migrated from D6 but still experiencing bugs (like #1309542: D7 core File field is not handled by Inline) which may or may not be related to this one about dependencies), I'm not the best person the change the status.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Fixed

Looking at the current code, this is already fixed on 7.x-1.x

Status: Fixed » Closed (fixed)

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