upgrade to 6.x

kdebisschop - June 10, 2008 - 04:09
Project:WYMeditor
Version:5.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

This a patch that gets wymeditor working in 6.x - it is not really the way I'd plan to go forward, it was just what I hacked out a few weeks ago when the project seemed leaderless and I needed a running editor for 6.x.

If people want to use this to get basic functionality for wymeditor in 6.x, I have this deployed on 2 6.x sites and it does work. But in truth, the patch mixes too many changes from coder and 6.x deployment and wymeditor 0.5a upgrade - it really is not a good candidate for review.

Once I can coordinate with the new maintainer, I'd suggest that we take the process in a set of smaller steps (he's already done the 0.5a upgrade) so that the patch cane be reviewed.

Anyway, to make his work:

- apply the patch
- add the .info module
- if you do not have the most recent code installed, get 0.5a from wymeditor and expand it into the wymeditor subdirectory of the module

#1

egfrith - June 12, 2008 - 10:32

I can't see the patch attached to this issue; could you try posting it again please, kdebisshop?

#2

kdebisschop - June 15, 2008 - 21:14
AttachmentSize
wymeditor-268731-17.patch.gz 6.81 KB

#3

ArjanLikesDrupal - June 24, 2008 - 22:28

Any developments on the plans for the port to 6.x branch? New co-maintainers?
Maybe I'll try WYMeditor on my D6 test site with this patch, but i'd like to know if there will be a proper D6 release anytime soon so I can use it when my new site is ready.

#4

zmove - June 25, 2008 - 07:27

subscribing

#5

yhager - June 25, 2008 - 12:30

I have applied the patch successfully and followed the rest of the installation instructions, but can't install a new profile.

The page 'admin/settings/wymeditor/profile/add' results in a 'not found' error.
Also the pages 'admin/settings/wymeditor/container/add', admin/settings/wymeditor/container/edit/3' return with the same error.

I looked at the code and it looks sane, I cleared up the cache using the UI and also truncated cache_menu, cache_page and cache tables.

Any ideas?

#6

ArjanLikesDrupal - July 3, 2008 - 00:58

kdebisschop, which version is this patch against? I assumed your patch was against 5.x-1.x-dev, but it didn't work for me, most hunks failed. Also tried to patch against other versions, no luck.

#7

yhager - July 8, 2008 - 15:17

@ArjanLikesDrupal, I think I used the HEAD version for the patch. It applied cleanly.

#8

ArjanLikesDrupal - July 8, 2008 - 16:41

Ah, should have tried that of course. Applied patch correctly to head, installed without errors. But then going to admin > settings > wymeditor > add profile leads to a fatal error:

Fatal error: Maximum function nesting level of '100' reached, aborting! in ~/includes/database.inc on line 210

any ideas?

I noticed there is now a 6.x-2.0-dev version. Cannot install it correctly though, lots of foreach() errors.

#9

zmove - July 15, 2008 - 12:21

Hi,

I tried the 6.x version too, have a lot of errors too, I think it is not ready to test ATM.

In addition, when I tried to desactivate it, I get a PHP error : passed variables by references, don't remember exactly...

I had to manually set the status to 0 in the system table in the database.

#10

Slim Pickens - July 28, 2008 - 02:11

kdebisschop - any chance of zipping up your working module folder and attaching it here please?

I've messed around with the patching and can't even find the right file to patch.

It's a great concept for editing, and I have a new site with a client completely unused to working with CMS. WYMeditor would be ideal, if it were working and usable.

Thanks.

#11

fall_0ut - October 8, 2008 - 19:58
Status:needs work» closed
 
 

Drupal is a registered trademark of Dries Buytaert.