Doesnt work with CCK

nbchip - November 11, 2008 - 05:42
Project:BookMadeSimple
Version:5.x-1.3
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

1.0 Version of module doesnt work properly with CCK. In content type settings when i check to Autmaticlly create new book root and then create content all the CCK field dont get properly saved...
1.3 gives white page on module enable, and on Auto create book main page...

#1

MarcElbichon - November 12, 2008 - 09:18

Could you delete all lines in book_made_simple.module after last } (with ?> line) and retry ?
If it doesn't work, try to increase server's memory settings.
May be http://drupal.org/node/158043 problem.

#2

nbchip - November 13, 2008 - 02:47

hm, im not too sure what u want me to delete (there are no closing php brackets ) i also deleted by reference sign & that was in front of &$form and others, but nothing helped...
Also there is no problem with memory, its on localhost with limit of 256mb
First i get white page, on refresh it works... so there is probably some whitespace issue

#3

nbchip - November 13, 2008 - 02:29

this is the error in drupal log

Cannot modify header information - headers already sent by (output started at D:\Program Files\xampp\htdocs\sites\all\modules\book_made_simple_5.x_1.0\BookMadeSimple\book_made_simple.module:142) in D:\Program Files\xampp\htdocs\includes\common.inc on line 314.

#4

MarcElbichon - November 13, 2008 - 12:07

Can you try with the last release ?

#5

nbchip - November 13, 2008 - 15:28

It is the last release , i just copied it in folder from previous... , u can see from path
\book_made_simple_5.x_1.0\BookMadeSimple\

#6

MarcElbichon - November 13, 2008 - 16:47

Sorry to be insistent but do you use 5.x-1.4 release ?

#7

nbchip - November 13, 2008 - 17:09

Sorry i just noticed the 1.4 version. now the module gets enabled fine. But still when i enable anything in CT settings, i get white page with form array printed and in that i see
node_type_form [book_made_simple_auto_main_page] => 1 [book_made_simple_add_types] => 0
And still the CCK fields dont get properly saved when creating Node of that CT

#8

nbchip - November 13, 2008 - 17:26

Hm, i renamed every occurrence of book_made_simple to xbook_made_simple so some of its functions get executed at the end of page submission proces... Now it works with CCK ! still i get White page on CT settings...

#9

MarcElbichon - November 14, 2008 - 13:11

I'm confuse. I forgot to delete debug trace.
Can you delete line 87 in book_make_simple.module.
For CCK, you're right, weight of the module must be set to a high value.
To do this, you can update mysql table by running :

update <yourschema>.system set weight=10 where name = 'book_made_simple';

OR

Retrieve 1.5 release at http://code.google.com/p/mmr-drupal-modules/downloads/list.
Run update.php script to correct weight of module

#10

nbchip - November 14, 2008 - 19:24

I was suprised to see that everything works fine , for now ;)
Good Work, and thx for fast responses!
One more thing, uninstall doesnt remove all references to book_made_simple in variables table, there are some left "book_made_simple_(content type name)", why is that?

#11

MarcElbichon - November 15, 2008 - 13:27

Right.
This is corrected in the last release.
Thanks for your contribution.

#12

MarcElbichon - November 17, 2008 - 10:18
Status:active» closed

#13

p_alexander - January 7, 2009 - 00:24

FYI, this change makes BookMadeSimple fire after the token module, which led to problems when using a token with Pathauto to create aliases for nodes. Changing the module weight to 1 seems to fix the issue, and content module is weighted at 0 so it should have the same affect there.

#14

MarcElbichon - January 9, 2009 - 18:15

With the lastest version from http://code.google.com/p/mmr-drupal-modules/downloads/list
and after running update.php , weight will be modified.

#15

p_alexander - January 14, 2009 - 20:55

It looks like it's still setting the weight to 10, which doesn't work with Pathauto. A setting of 1 should fix the CCK issue and still function when used in combination with Pathauto.

#16

MarcElbichon - January 16, 2009 - 08:41

This will work with pathauto but not with modules that name is alphabetically smaller than bookmadesimple.
So, i suggest you to change manually the weight of bookmadesimple.

 
 

Drupal is a registered trademark of Dries Buytaert.