Inline is very useful module. Updating HEAD for Drupal 6x beta
will be very appreciated.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Title: Update INline for Drupal6x » Port Inline to 6.x

Agreed. Who wants to take up the ball?

sun’s picture

Category: feature » task

This task depends on Inline API to be ready.

SiliconMind’s picture

FileSize
7.53 KB
551 bytes

Here is quick (and dirty) patch for inline-5.x-1.1 to make it work with drupal 6.x
add new file inline.theme.inc and apply attached patch.
you'll also need ImageCache patched for drupal 6.x, so check this thread: http://drupal.org/node/232624

eaton’s picture

Status: Active » Needs review

Here's a version of the patch that removes the notices, and includes the theme.inc file without requiring a separate download.

eaton’s picture

Version: 6.x-2.x-dev » 5.x-1.1

Hmmmm. On the other hand, attaching the patch file doesn't seem to work. Oh, well.

eaton’s picture

FileSize
13.06 KB

That's more like it.

sun’s picture

Status: Needs review » Needs work

- AFAIK, menu item descriptions should not be t()'d
- Why is that change?

 function inline_nodeapi(&$node, $op, $arg) {
-  if (!is_array($node->files)) {
+  if (empty($node->files) || !is_array($node->files)) {

- This also looks like a logical change to me - why?

-          $node->teaser = _inline_substitute_tags($node, 'teaser');
-          $node->body = _inline_substitute_tags($node, 'body');
+          if (!empty($node->teaser)) {
+            $node->teaser = _inline_substitute_tags($node, 'teaser');
+          }
+          if (!empty($node->body)) {
+            $node->body = _inline_substitute_tags($node, 'body');
+          }

- CVS Id tag is missing on first line in inline.theme.inc

Other than that, the patch looks good. However, it would be nice if those coding-style changes could be avoided (leaving code clean-up to another patch against both D5 and D6 afterwards).

eaton’s picture

Both of the changes you noted were causing notices to be thrown when nodes were loaded -- uninitialized variables being referenced, etc. Apologies for the whitespace -- my editor was set to automatically strip trailing spaces and that showed up in the patch without my realizing it. I'll roll a new version without that. :)

sun’s picture

Let's deal with all coding style and E^ALL fixes in #273489: Code clean-up, thus limiting this patch to required changes for D6 only.

sun’s picture

#282173: Drupal 6 port has been marked as duplicate of this issue; contains a patch, too.

eaton’s picture

FileSize
9.36 KB

Re-rolled version of the patch without the E_ALL fixes or whitespace corrections. Hopefully it will be committed at some point.

sun’s picture

Version: 5.x-1.1 » 6.x-2.x-dev
Category: task » feature

Well, I think we're close...

- Menu item description still contains t().
- inline.theme.inc misses a CVS Id comment on second line of the file.

Also, when trying to apply this patch:

6 out of 8 hunks FAILED
eaton’s picture

Just an FYI that I'll be using the earlier version with the E_ALL fixes on my own blog; others are welcome to continue the porting work.

sun’s picture

Assigned: Unassigned » sun
Status: Needs work » Needs review
FileSize
10.12 KB

ah, well... it seems that all of you are working with an extremely out-dated version of Inline (> 1 year). That explains, why all posted patches do not apply at all.

Please test attached patch - which should apply cleanly against the latest development snapshot.

koorneef’s picture

Tried the patch on: inline-5.x-1.x-dev.tar.gz
Patch applied except for the .info file, manually editted this file.
Could enable the module, but when trying to administer module got a PHP error:
Failed opening required 'sites/all/modules/inline/system.admin.inc'

Inline shows up in inputfilter admin pages.

No further testing done.

sun’s picture

Hm. I do not think that this error is related to Inline module and this patch, because there is no reference to system.admin.inc in this patch.

sun’s picture

Status: Needs review » Fixed
FileSize
9.85 KB

Committed attached patch.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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