if you have books enabled
it will display
books at mypage
book1Arraybook2Array ...

i have sloved
with in line 268 269 at site_map.module

268      $output1 = ($data['below']) ? _site_map_menu_tree_output($data['below']) : '';
269      $output .= render($output1);

maybe not the best soulution but it works

greetings

CommentFileSizeAuthor
#1 site_map_book_bugfixes.patch9.14 KBfrjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

frjo’s picture

frjo CreditAttribution: frjo commented
Version: 7.x-1.0-beta1 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
site_map_book_bugfixes.patch9.14 KB

Thanks for reporting the broken support for books in Site map.

I have committed the attached patch to 7-dev and I believe that fixes this bug. I also re added the option to "Show books expanded".

The 7-dev tarball will take some hours before it's rebuilt.

Please test and report back here.

Kisugi Ai’s picture

Kisugi Ai CreditAttribution: Kisugi Ai commented

with devel but its more a Drupal issue

Strict warning: Only variables should be passed by reference in _site_map_books() (line 317 of /sites/all/modules/site_map/site_map.module).

317     $output1 = _site_map_menu_tree_output($tree);
318     $output .= drupal_render($output1);
frjo’s picture

frjo CreditAttribution: frjo commented

Thanks for the followup. Is the patch working ok otherwise?

Committed the following to 7-dev that should take care of the warning.

        if ($books_expanded) {
          $tree_output = _site_map_menu_tree_output($tree);
          $output .= drupal_render($tree_output);
        }
Kisugi Ai’s picture

Kisugi Ai CreditAttribution: Kisugi Ai commented

yepp the other works at the moment fine.
works under Dru8 to

frjo’s picture

frjo CreditAttribution: frjo commented
Status: Needs review » Fixed

Thanks reporting and testing this!

Status: Fixed » Closed (fixed)

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