[Fixed]: Missing Closing H2 Tag Possibly Renders Update Output Unreadable [Fixed]: Missing Closing H2 Tag Possibly Renders Update Output Unreadable
 

News:

cpg1.5.48 Security release - upgrade mandatory!
The Coppermine development team is releasing a security update for Coppermine in order to counter a recently discovered vulnerability. It is important that all users who run version cpg1.5.46 or older update to this latest version as soon as possible.
[more]

Main Menu

[Fixed]: Missing Closing H2 Tag Possibly Renders Update Output Unreadable

Started by onthepike, June 17, 2010, 12:32:25 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

onthepike

Regarding this thread: Need Help Changing Theme

I was able to reproduce and provide an example of the update.php output rendering white text against the default grey table data background cell. See link above for details and visual examples attached below.

In searching for an explanation, I happened across an unclosed h2 tag in update.php (Revision: 7613):


   // Update table prefix
   $sql_query = preg_replace('/CPG_/', $CONFIG['TABLE_PREFIX'], $sql_query);

   $sql_query = remove_remarks($sql_query);
   $sql_query = split_sql_file($sql_query, ';');

   ?>
       <h2>Performing Database Updates<h2>
       <table class="maintable">


Changing the line

<h2>Performing Database Updates<h2> to (obviously) <h2>Performing Database Updates</h2>

Resulted in my no longer having the ability to duplicate the issue. Therefore, I am going to attribute this intermittent error (which is most likely browser-setting respective, since it's not visible in all applications) to the open tag if and until the issue presents itself once again.

I hope I have not over-stepped by posting here. There didn't seem a more appropriate board within 1.4 Support since this issue does deal specifically with a typo in an html tag.

Also, while this issue would generally be considered cosmetic in nature, it is apparently easily solved by closing an html tag.

Thank you for your time.

Jeff

Αndré