Extension talk:MediaList

Add topic
From mediawiki.org
Latest comment: 15 years ago by Ozz in topic Option table do not work

Well ... this is my first extension ... so be merciful and contructive in your criticism in case I screwed something. ;-)
--Wolverine 14:29, 21 December 2007 (UTC)Reply

Option table do not work[edit]

Hi, great extension! cuurently if choose "table" the extension gives nothing back. It works only with

<medialist/>
<medialist style="simple">Informations about all files used by this article</medialist>

--Ozz 18:22, 28 July 2008 (UTC)Reply

Sorry ... haven't checked here for quite some time.

curently if choose "table" the extension gives nothing back.

Hmm ... that's strange ... the mechanism is pretty straighforward and it work's pretty well in our wiki here. Which version do you use? We currently still have 1.11.00 in use.

The way the table is build is pretty simple, so I doubt that it would conflict with changes in CSS or something like that. But if you get nothing at all back the recognition of the style attribute table seems to work, 'cause in all other cases the simple list output is created, which works for you as well.

You might want to replace the outputMedia function for debugging purposes with the following, to see if tables work at all:

  function outputMedia($filelist)
  {
    $output = '';
 
    if ($this->settings['style'] == "table")
    {
      $output .= '<h3>medialist: Test for a standard table</h3>';
      $output .= '<table><tr><td>Standard table</td><td>second column</td></tr></table>';
      $output .= '<h3>medialist: Test for a medialist table</h3>';
      $output .= '<table width="100%" border="1" cellspacing="0"><tr><td>medialist table</td><td>second column</td></tr></table>';
    }
    else
    {
      $output .= '<h3>medialist: No table choosen</h3>';
    }
 
    return $output;
  } // end of outputMedia

--Wolverine 12:25, 25 September 2008 (UTC)Reply

Hello Wolverine, thanks for the hint. In the meantime i switched to MW 1.13 and now is works correctly. --Ozz 12:07, 26 September 2008 (UTC)Reply