Port Inline to 6.x

Agreed. Who wants to take up the ball?

This task depends on Inline API to be ready.

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

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

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

That's more like it.

- 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).

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. :)

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.

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

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

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

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.

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.

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.

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.

Committed attached patch.