Title depends on Drupal core 7.15 but this is documented only on the front page. We need to add an explict dependency in the .info file (see http://drupal.org/node/1626346).

CommentFileSizeAuthor
#3 1869228-3-followup-revert-1519930.patch2.47 KBdrzraf
#1 title-dependency-1869228-1.patch387 bytesplach

Comments

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Active » Fixed
StatusFileSize
new title-dependency-1869228-1.patch387 bytes

Committed and pushed the attached patch.

Status: Fixed » Closed (fixed)

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

drzraf’s picture

drzraf commented
Status: Closed (fixed) » Active
StatusFileSize
new 1869228-3-followup-revert-1519930.patch2.47 KB

you may have noticed that Debian provides Drupal 7.14. This could be so for some time.
Don't you think it would be useful to offer a way to use title for drupal7 package users ?
This could be a patch, a wrapper for entity_language, ... or anything else.

For now this attachment reverts patch from #1519930: Use the upcoming entity_language() function to determine the entity language. It's not intended to be merged as-is, but a support for discussing pre-7.15 compatibility which was dropped (apparently) without much discussion.

This patch might or might not be enough for Debian as the very same issue affects entity_translation and both modules are often used together.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Active » Closed (fixed)

Sorry, I'm not willing to support pre 7.15 code: other modules might be interested in providing a fallback, since they have optional Title/ET integration. In this case pre 7.15 code would be malfunctioning at best, thus I don't see the point of supporting it. You won't get a fatal, but things won't work in more subtle ways. I don't see the gain.

  • Commit 11d058c on 7.x-1.x, workbench by plach: 
    Issue #1869228 by plach: Updated system dependencies.