Drupal 7 port of Embedded Widgets

any news on this?

Time has gone by without any answer. Can we get at least a rough time frame?

I haven't had time recently to work on a D7 port. I think that a D7 version should have a dependency on the Ctools module. It will provide the following features/APIs that Embed Widgets currently implements itself:
* Content -- pluggable content types used as panes in Panels and other modules like Dashboard. This will let us use the Ctools content API to embed content such as blocks, views, nodes, etc. in widgets instead of reinventing the wheel.
* Plugins -- tools to make it easy for modules to let other modules implement plugins from .inc files. Use the Ctools plugin system to allow plugins for different types of widgets (iframe, javascript, flash) and different providers.
* Contexts -- the notion of wrapping objects in a unified wrapper and providing an API to create and accept these contexts as input. We may be able to use this to determine when we are in a widget and when we are not.

Exportables and Object caching may also be useful.

How many hours would the D7 port likely take?

It seems like Drupal 7 is nearly completely dependent upon Ctools - not just this project, but many others - and is suffering without it.

here's a start. will post an updated version tomorrow (hopefully fully working).

note that this version doesn't include any dependency or work using ctools.

OK, this works now, although I've only tested with embedding nodes thus far.

use this instead (adds a new theme function for the block content).

and this adds a wrapper to the block content...

I haven't addressed @BenK's suggestions yet. But this one takes care of some sidebar issues when embedding a page.

subscribing

Hi All,

@aaron, thanks for the patch. It's just what my company needs. Sadly, this is my first attempt to patch a Drupal module, and it's not going well. I hope you guys can help.

Doing this on a Mac, w/ Mac OS 10.6.6, in Terminal using the patch command. The Mac is my laptop, not the server running Drupal.

1. I've downloaded the 6.x-2.x-dev version of Embed Widgets to a test directory on my Mac.
2. I place the patch file in the embed_widgets directory.

In Terminal, I go to the embed_widgets directory and run the following command:

patch < embed-widgets-d7-port-795226-14.patch  --dry-run --verbose

Things get off to a bad start. Here's the very start of the output…

Hmm...  Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|diff --git a/embed_widgets.info b/embed_widgets.info
|index 53da148..977f54b 100644
|--- a/embed_widgets.info
|+++ b/embed_widgets.info
--------------------------
Patching file embed_widgets.info using Plan A...
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file embed_widgets.info.rej

(Continuing the same output stream) Things seem to go better with the embed_widgets.install file, and with the module itself:

Hmm...  The next patch looks like a unified diff to me...
The text leading up to this was:
--------------------------
|diff --git a/embed_widgets.install b/embed_widgets.install
|index 898390f..30054d4 100644
|--- a/embed_widgets.install
|+++ b/embed_widgets.install
--------------------------
Patching file embed_widgets.install using Plan A...
Hunk #1 succeeded at 1.
Hunk #2 succeeded at 87.
Hunk #3 succeeded at 111.
 [...]
Patching file embed_widgets.module using Plan A...
Hunk #1 succeeded at 1.
 [...]
Hunk #26 succeeded at 903.

Then things get weird and come to an early & abrupt end:

Hmm...  The next patch looks like a unified diff to me...
can't find file to patch at input line 964
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/embed_widgets_ui/embed_widgets_ui.info b/embed_widgets_ui/embed_widgets_ui.info
|index 3e33ec5..d8d83cf 100644
|--- a/embed_widgets_ui/embed_widgets_ui.info
|+++ b/embed_widgets_ui/embed_widgets_ui.info
--------------------------

...After which the patch process drops me at the prompt: "File to patch: ".

Line 964 specifies the embed_widgets_ui.info file, which is where it should be. Here's line 964:

+++ b/embed_widgets_ui/embed_widgets_ui.info

I'd know where to look next if it failed completely, but with this mix of success & failure for different files, I'm at a loss.

Anyone know what might be going on?

subscribing - can't wait for a block or view display.

I believe if i can get one of those working i will get what i need. I will try to play around with this myself. no promises though.

Hi all,

I cant patch it either, can someone zip and upload a patched version please.

Kind Regards,

Aaron

@Syph

try applying the patch with git

git apply -v [patchfile]

ty Jackinloadup, has anyone made any progress with block or views?

I made an attempt but figured out how to get page to do what I needed. As far as I can see node and page are the only ones working. Just an FYI for everyone else at this point.

im still not able to patch it for some reason. ill keep trying and let you know what happens

subscribe

Can anyone post a summary of how far along this module is for a stable D7 release and what's holding it up?

sub

subscribe

sub

+стопицот

Note that there is a D7 dev release of Web Widgets, http://drupal.org/project/web_widgets, which seems to provide some similar functionality. I haven't tested it.

Here is a patch that cleaned up a lot of the issues identified in #11. This should make it so that block widgets are working. still working on others, but here is a better start.

This patch includes stuff from the patches above so apply this to 6.x-2.x.

I have tried applying the patch from #33 to embed_widgets-6.x-2.x-dev and embed_widgets-6.x-2.0-alpha2 using bot "git apply " and "patch -p1 < "

The best result seemed to be with 6.x-2.0-alpha2 and "patch -p1 < " but one file still failed to apply.

patching file embed_widgets.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file embed_widgets.info.rej

Thanks all for the work on updating.

A couple of points from a quick scan:

+++ b/embed_widgets.module
@@ -9,13 +9,13 @@
+    'description' => 'The embedded file. ',

There are a lot of mistaken additions of a space between a period and an end quote, as here. Maybe a cut and paste error?

+++ b/embed_widgets.module
@@ -40,55 +40,61 @@ function embed_widgets_menu() {
+  global $_embed_widgets_mode;

Should be replaced with a drupal_static() call.

This patch is a hard to review partly because it mixes in a lot of code styling updates.

Ok, got it to work after cloning the git repo

git clone --recursive --branch 6.x-2.x http://git.drupal.org/project/embed_widgets.git
cd embed_widgets

and applying the patch

wget http://drupal.org/files/embed_widgets-d7.patch
git apply embed_widgets-d7.patch

The widget preview seems to work but can't find a way to view the "Share this Widget!" link to show up using the "Append widget link to blocks." option.
Also, it would be nice if the widget was disabled by default. I only want it on one block.

Sorry about the code styling mixed with upgrades, I kind of try to fix things as i see them. I havent had much time to work on this as other projects have needed my attention, but those other projects are winding down, and i will be picking this up again shortly.

Just confirming I've used the the #14 patch to embed a webform in an iframe.

I'm still having problems with the views embedded widgets as described in #11 and following.
Did anybody solve someway?
Many thanks!