modifyalb.php problem

[rosbif]

Not sure when this started but suddenly my 'Update album' page no longer works. The button 'Update album' links to

Quote::

http://www.lasourcetranquille.com/'.getlink(

which obviously goes nowhere. I don't think I've changed any files - the code at line 390 of modifyalb.php (version 9.6) says

PHP:

<tr><form method="post" action="'.getlink("&amp;file=db_input").'" enctype="multipart/form-data" accept-charset="utf-8">

I'm sure it used to work!

[unigene]

I guess you use IE9. Click the 'Compatibility View' button and everything will work again. I had the same issue.

[rosbif]

Ha! That simple. Any idea what M$ have done to break it? (and thank you!)

[InspectorClueNo]

the code is parsed by the server, not by browsers ... how weird.

are you sure that changing compatibility mode fixed the issue?

[rosbif]

I was going to say absolutely - but I've just gone back to try it and it is now working without compatibility mode.. How bizarre is that? I had to view it with compatibility mode to get it to work, and now I don't need that mode.

Odd

[layingback]

rosbif wrote:

I was going to say absolutely - but I've just gone back to try it and it is now working without compatibility mode.. How bizarre is that? I had to view it with compatibility mode to get it to work, and now I don't need that mode. Odd

Because the fully resolved link is now in your browser cache?

[InspectorClueNo]

nope ... it's parsed by the server and saved into website/cache.tpl ... probably a glitch?

[rosbif]

It is doing it again - like lb said I presume it gets the correct link from the cache, but when the correct link is not in the cache it comes up with an error. Clicking the compatibility link and it works again.

[rosbif]

This is really bugging me. Sometimes it doesn't work with compatibility mode on either - I have to select it off, then on again, sometimes twice for it to work. Why could that be?

[rosbif]

Other files in coppermine seem to have the same problem in IE9 - this wierd getlink error. Would love to get to the bottom of it...

[InspectorClueNo]

If you using older version of DragonflyCMS:
in modifyalb.php find a spot for:

PHP:

$cpgtpl->assign_var('U_DB_INPUT', getlink('& amp;file=db_input'));

in the template file change:
from

Code::

action="'.getlink("& amp;file=db_input").'"

to

Code::

action="{U_DB_INPUT}"

---

instead if you using the most recent version of DragonflyCMS without any additional php code you simply change your template to read:
from

Code::

action="'.getlink("& amp;file=db_input").'"

to

Code::

action="U_('& amp;file=db_input')"

note: "& amp;" should be without any space, it does have one here for display only

[rosbif]

Thanks Nano. Which template file are you referring to?

[NanoCaiordo]

change /modules/coppermine/modifyalb.php start at line 377
from:

PHP:

<?php # below starts line 377 $album_lb = alb_list_box(); $chset =_CHARSET; echo ' <tr> <td colspan="2" class="tableh1"> <table width="100%" cellpadding="0" cellspacing="0" border="0"> <tr> <td class="statlink"><h2>'.UPDATE.'</h2></td> <td align="right">'.$album_lb.'</td> </tr> </table> </td> </tr> <tr><form method="post" action="'.getlink("&amp;file=db_input").'" enctype="multipart/form-data" accept-charset="utf-8"> <input type="hidden" name="event" value="album_update" /> <input type="hidden" name="aid" value="'.$album.'" /> '; create_form($data);

to:

PHP:

<?php # below starts line 377 $album_lb = alb_list_box(); $chset =_CHARSET; $u_db_input = getlink('&amp;file=db_input'); echo ' <tr> <td colspan="2" class="tableh1"> <table width="100%" cellpadding="0" cellspacing="0" border="0"> <tr> <td class="statlink"><h2>'.UPDATE.'</h2></td> <td align="right">'.$album_lb.'</td> </tr> </table> </td> </tr> <tr><form method="post" action="'.$u_db_input.'" enctype="multipart/form-data" accept-charset="utf-8"> <input type="hidden" name="event" value="album_update" /> <input type="hidden" name="aid" value="'.$album.'" /> '; create_form($data);

Thanks for thanking me in advance ?!
However this is a revised version of InspectorClueNo's idea since modifyalb.php does not use any templates yet, in any case thanks ClueNo.

rosbif, let us know if this helps.

[layingback]

Actually there is a template in rosbif's case - reDesign 3. But fixing in it in the code surely saves fixing in each and every template.

I guess we all need to apply this fix if we expect IE9 users?

So could either of you who explain what is going wrong here with IE9, please? (Or is it just a(nother) case of shuffling the code to avoid an unidentified PHP source code problem?)

TIA.

[rosbif]

Nano,

It doesn't make any difference. I thought perhaps I had to change the similar getlink at line 338 - so I edited this to:

PHP:

if ($db->sql_numrows($result)) { $u_modifyalb = getlink('&amp;file=modifyalb,0,1'); $lb = '<form method="post" action="'.$u_modifyalb.'" enctype="multipart/form-data" accept-charset="utf-8"><input type="hidden" name="name" value="'.$module_name." />

This link works correctly (it's the change album link) with either the original getlink or the version above.

For the second button IE9 is still showing a link to ".com/'.getlink(".

The source shows (correctly)

Code::

</tr> <tr><form method="post" action="coppermine/db_input.html" enctype="multipart/form-data" accept-charset="utf-8">

so it just appears to be IE9 interpreting it incorrectly.