per node type settings available?

This is a good suggestion.
Try to do this but i'm very busy at this time so i don't know when i can do this.

If somebody wants to work on it .............

Hi,

Can you test this version ?
Rename the file to book_made_simple.module and copy it in the module directory.

In the content type form, in BookMadeSimple section, i added two lists. One for content-types allowed for this content type and another for content-type allowing this one.

This selection will overwrite content types list of the BMS settings. If no selection has been found for a content-type, those selected in BMS settings will be shown.

Are you OK with this ?
Could you test for me ?

Be careful. For a content-type, if you allow the same content-type as child, don't forget to add it too in the allowing content-type list.

Yeah, this works fine!

But some hints and suggestions:

• The selection box "Add child page" only works with javascript enabled. So you better display this as an unordered list that can be formatted with css. That's for accessibility reasons.
• The text "Select the content-types allowing this content-type as child" should be changed to "Select content-types allowed as parent" (easier to understand)
• Selection boxes with multiple select are not very user friendly, as you have to press the ctrl-key when clicking on an item. If you forget this, you will only select one item. As these lists may not grow that big, you can choose to display a list of checkboxes instead. So everybody knows, multiple selection is possible and thus it is more user friendly.
• You can omit the current node type in the "parents" list to not confuse the user (as I was at first, then remembered your warning). If I select the current node type as possible child, this node type is also a possible parent. And if I do not select this node type as a possible child, it is not a possible parent ,too.
• Can you please find a way to remove the default "Add child page" link? Modifying a css file is not everybodies darling.
• Can you please add a link to reorder the current book admin/content/book/[bid] for privileged users? I think, the way provided by the book module, to link to all books, is not very user friendly.
  

1. There's a theming function to do this.
   In your template.php, create this function :
   
   <?php
// First argument is array of allowed content types
// Second argument is the node
function yourtheme_add_child_book_content_types($allowedTypes, &$node) {
       $html = "<ul>";
       foreach ($allowedTypes as $type => $name ) {
        $isCleanUrl = variable_get('clean_url', 0);
        $newUrl = base_path() . "node/add/" . str_replace('_', '-',$type) . "/parent/" . $node->nid;
        if (! $isCleanUrl) {
            $newUrl = base_path() . "?q=node/add/" . str_replace('_', '-',$type) . "/parent/" . $node->nid;
        }
        $html .= "<li><a href='" . $newUrl . "'>"  . $name ."</a></li>";}

        return $html . "</ul>";
}
?>
2. OK to change
3. Select changed to checkboxes
4. Current content-type is removed from parent content-type list
5. Added a checkbox in BMS settings to hide Add child page link
6. Could you be more explicit ?

Could you test this new version ?

A little correction pour FireFox

ad 1) That might be the case. But if you do not use an own theme? Create one to implement this function? And what does theming have to do with BMS?

ad 6: I mean something like this:

Sorry, but I am not a php programmer.

Ok, looks much better with checkboxes ;)

But what does "Allow content type as child of default book." mean? Either I can select this node type as child of "book" or not. I know, you need "book" for your "Add child page" selection box, as it needs at least two entries to work correctly. And the first entry "Add child page" always links to "book".

Another caveat with this selection box:

• select the new child type to create
• press the "back"-button on your browser
• select "Add new child" in the selection box: create book page appears!

How do you want to explain this behavior to normal users? And, as I wrote earlier, you should not rely on javascript where not necessary. Either provide a javascript and a link version or a link-only version.

What about this:

(This was a modification of your released module)

Allow content type as child of default book is for compatibility.

In BMS settings, you can select content-type to add to book. This will be default values for content-type with no child defined.

This means that this content-type will be allowed for all content-type (even new content-type) without have to add it in form setting.

To modify selection,you must theme the function. See theming with Drupal for more informations

Added to release 2.0

I have had a quick look at that new release available at your google page. You seem to have added some nice features. But your code is not clean enough to provide an other release than a dev release!

• Different indentations with spaces or tabs. Depending on editor settings tabs are expanded to a different number of spaces. So the code does not look nice.
• unresolvable references like in t('You can also defined childs by content-type in <a href="http://localhost/drupal-6.6/admin/content/types/list">content-type settings.</a> in the BookMadeSimple section')
• missing t() like in '#title' => 'Auto create book main page' and some other places.

I will also encourage you to find a way to solve your problem with the firewall as you break the update status for your module! Also issues for your module can not be filed with the correct version number on drupal.org. Perhaps you can ask someone else to do this job for you?

Nevertheless, you did a great job! In my opinion, this job should have been done by the developers of the book module themselves!

I'm completly agree with you for the last remark of you post (lol) !!!!

#1 - Changed tabs to chars
#2 - Corrected
#3 - Corrected. Added t() function to values of dropdown listbox
#4 - I'd like but Drupal don't want to open 2403 port. Would you want to publish code for me ?

Sorry, I can't help you in publishing the code. I have currently no experience in that manner. Please ask someone else :(

This applies to your other modules, too.