Any volunteer to port this great module to Drupal 7? I would be happy to contribute testing and an updated README.txt file.

CommentFileSizeAuthor
#8 imce_wysiwyg_670818_8.patch489 bytesfrancewhoa
#9 imce_wysiwyg-HEAD.tar_.gz4.11 KBfrancewhoa
#6 error.png25.85 KBfrancewhoa
#1 imce_wysiwyg-HEAD.patch584 bytessun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Active » Fixed
StatusFileSize
new imce_wysiwyg-HEAD.patch584 bytes

Thanks for reporting, reviewing, and testing! Committed to attached patch.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

francewhoa’s picture

francewhoa
he/him
Primary language French
Location Sept-Îles, Québec, 🇨🇦
commented

Awesome. Thanks sun. I'll test and post my result here.

If someone else wants to contribute testing here is the direct link to Drupal 7 dev version (7.x-1.x-dev) http://drupal.org/node/670850

Status: Fixed » Closed (fixed)

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

ccoppen’s picture

ccoppen commented
Status: Closed (fixed) » Active

Apparently the latest version doesn't work with Drupal 7.x. I'm getting the following on the modules tab in the latest version of Drupal 7.x

This version is not compatible with Drupal 7.x and should be replaced.

bmagalhaes’s picture

bmagalhaes commented

I´m having the same problem.

Bruno de Oliveira Magalhães
www.7links.com.br
www.habitus.com.br

francewhoa’s picture

francewhoa
he/him
Primary language French
Location Sept-Îles, Québec, 🇨🇦
commented
Status: Active » Needs work
StatusFileSize
new error.png25.85 KB

Confirming that patch in #1 doesn't work. Drupal 7 says This version is not compatible with Drupal 7.x and should be replaced. Find attached screenshot to clarify.

Tested with
Drupal 7.0-alpha3
imce_wysiwyg 7.x-1.x-dev (2010-Feb-21)
imce 7.x-1.0-alpha1 (2010-Mar-17)
wysiwyg 7.x-2.x-dev (2010-Apr-07)

I tried both the patch in #1 against HEAD imce_wysiwyg 7.x-1.x-dev (2010-Feb-21) and imce_wysiwyg 7.x-1.x-dev (2010-Feb-21) as is without the patch with same result.

forngren’s picture

forngren commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Category: task » bug
Status: Needs work » Needs review

By adding this line to imce_wysiwyg.info:

files[] = imce_wysiwyg.module

you can activate the module. See http://drupal.org/node/224333#registry.

I can't provide a patch-file atm, but it's very simple to apply. Please test and mark RTBC if it works.

francewhoa’s picture

francewhoa
he/him
Primary language French
Location Sept-Îles, Québec, 🇨🇦
commented
StatusFileSize
new imce_wysiwyg_670818_8.patch489 bytes

Thanks forngren.

Patch is for 7.x HEAD.

Any volunteer to test it? I can't now.

francewhoa’s picture

francewhoa
he/him
Primary language French
Location Sept-Îles, Québec, 🇨🇦
commented
StatusFileSize
new imce_wysiwyg-HEAD.tar_.gz4.11 KB

Module with patch already applied. Any volunteer to test?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Fixed

Thanks for reporting, reviewing, and testing! Committed to HEAD.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

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