Last modified: 2009-02-20 10:56:57 UTC
buildSidebar() returns a raw string to be included in the sidebar. This doesn't work with the DumpHTML extension, because it expects the format normally returned by Skin::buildSidebar(), which is built like this (line 1869 in includes/Skin.php): $bar[$heading][] = array( 'text' => $text, 'href' => $href, 'id' => 'n-' . strtr($line[1], ' ', '-'), 'active' => false ); The Collection extension should be changed to return the same format.
It's allowed for extensions to insert raw HTML instead of an array, see e.g. line 366 in skins/MonoBook.php in customBox(): <?php } else { # allow raw HTML block to be defined by extensions print $cont; } ?> Anayway, thanks for reporting: I looked at the sidebar building code and discovered that wfMsg() is later called on the heading, so we finally have the same id p-create_a_book ID for all languages. This is fixed in revision 47027.
Thanks for the fast response! New bug submitted against DumpHTML: Bug 17586.
