I need the issu integration for a project that I'm starting. Project is going to run on D7. So I tried to port this module. I never build a module before so this was an intersting try.

Things that are alrdy working:
- I made an issuu file widget ==> WHOEHOEW!
- I can select the 3 formatters ==> WHOEHOEW!
- we have build the widget form, and included the aditional data into the file field ==> WHOEHOEW!

Things that don't work yet
- fetching the language list
- Add the extra issuu forms, (need help with that would like to do this my self but don't know how to do it, tried hook_element_info but without good results).
- fix the presave function
- DEPENDENCY ISN'T PORTED YET :( :( :(

Watch out for the debug messages ;)

Could you throw this code in a D7 branch?

CommentFileSizeAuthor
issuuD7port.patch34.73 KBaspilicious

Comments

marvil07’s picture

marvil07 commented
Status: Active » Needs review

Thank you very much for opening this with a patch!

I did not yet take a look, but will be trying to make some time space to do it, probably before end of January to work on D7 version of this module.

aspilicious’s picture

aspilicious commented

Any news on this? I need this in a few months...

fuse’s picture

fuse commented

+ 1

dgastudio’s picture

dgastudio commented

subscribe

marvil07’s picture

marvil07 commented

BTW my plan for 7.x: Make the module dependent on Media.

marvil07’s picture

marvil07 commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
pelicani’s picture

pelicani commented
Issue summary: View changes

This still appears to be active after 3 years.
We are going to implement the 7.1 version of issuu integration and need to apply this patch to upgrade hooks from 6 to 7.
However, this patch fails on the most important file, the module file.
Being so old, this must have been run on a much earlier version of this dev module.

The best I can think to do is manually update this patch, then create a new one to post here.
no?

peace,
michael

pelicani’s picture

pelicani commented
Status: Needs review » Needs work

we manually updated the patch, and tweaked some of the code.
the display functions are not implemented
after we look into them, i'll try and remember to post my patch here

marvil07’s picture

marvil07 commented

It's been a long time since I look at this code, but IIRC there is a 7.x-1.x branch in code which was mostly working since 2011. I do not really remember why I did not open a release, but code is there.
@pelicani: I would suggest you to start there.

mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
commented

There is also a 7.x-2.x-branch. This branch only contains media-integration (media_issuu module) if this is sufficient for your use case it might be better to collaborate on that branch.

pelicani’s picture

pelicani commented

I found the 7.1.dev code, but it wasn't working.
After I manually applied this patch, and added some output display, I got it working.
I could not find the 7.2.dev version.

Thanks for your quick comments.
This issue has been inactive for some time.
So far, I'm liking the integration.
But I agree, I'd like to work on the most recent so this module can continue to evolve.

peace,
michael

mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
commented

There is no (tarbal) release for the 7.X-2.X branch, you can access the code through a git clone, and change the branch. See https://drupal.org/node/714390/git-instructions/7.x-2.x/nonmaintainer
I'm not able to create a dev-tarbal release, i'll contact the main maintainer to hopefully get this fixed.

mallezie’s picture

mallezie
Primary language Dutch
Location Loenhout
commented
Status: Needs work » Fixed

The code in this patch was the base for the 7.x-1.x-branch.
The 7.x-2.x media branch now has dev-releases.

Status: Fixed » Closed (fixed)

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