The inspection claims "light" is always false. <% String color ; boolean light = true ; %> <logic:iterate id="reservation" scope="request" name="ccs.conflict.list"> <% if( light ) { color = " light" ; } else { color = "dark" ; } light = !light ; %> <tr> <td class="<%= color %>">...</td> </tr> </logic:iterate>