[DaveS]

Also in the install script (installation.txt) in the template modifications section you have too many closing TD tags I think.

<fieldset id="photobox" title="$vbphrase[photo_popup_my_photos]">
    <legend>$vbphrase[photo_popup_my_photos]</legend>
    <table cellpadding="4" cellspacing="0" border="0" align="center">
        <tr>
            <td class="smallfont" colspan="$vboptions[smcolumns]">
                       [<a href="#" onclick="window.open('$vboptions[bburl]/photo_popup.php?e=$editorid','fotos','scrollbars=yes,resizable=yes,width={$vbulletin->options['photo_popup_width']},height={$vbulletin->options['photo_popup_heigth']}'); return false" title="$vbphrase[photo_popup_insertlink]">$vbphrase[photo_popup_insert]</a>]
            </td>
            </td>
	    </td>
        </tr>
    </table>
</fieldset>