0008578: Wrong style class used in if ... else - MantisBT - Signup temporarily disabled due to spam

ID	Project	Category	View Status	Date Submitted	Last Update

0008578	mantisbt	webpage	public	2007-11-13 14:42	2008-04-19 04:10

Reporter	ShawnClark	Assigned To	vboctor
Priority	normal	Severity	trivial	Reproducibility	always
Status	closed	Resolution	fixed
Fixed in Version	1.2.0a1

Summary	0008578: Wrong style class used in if ... else
Description	The if has row-1 while the else has row-2. Either case should have row-1 for the class.
Tags	No tags attached.
Attached Files	account_page.php (4752) - patch.txt (403 bytes) Index: mantisbt/account_page.php =================================================================== --- mantisbt/account_page.php (revision 4752) +++ mantisbt/account_page.php (working copy) @@ -182,7 +182,7 @@ <?php } else { ?> <!-- Without LDAP Email --> <!-- Email --> - <tr class="row-2"> + <tr class="row-1"> <td class="category"> <?php echo lang_get( 'email' ) ?> </td> account_page.php (4752) - patch.txt (403 bytes)

ShawnClark 2007-11-13 14:51 reporter ~0016191	This might not actually be a bug after looking at it further. I was relating the alternating row style from the code below the if..else and didn't take into account the code above. There is another if..else above where the if has only two rows while the else has three rows.

ShawnClark 2007-11-13 16:46 reporter ~0016192	Optimal solution would be to count the rows displayed while they are being displayed and having a mod of the count to determine if row-1 or row-2 should be used.

giallu 2007-11-20 05:28 reporter ~0016248	That is exactly the purpose of the helper_alternate_class() function