Bad Code Green

[Stuart Hemming]

Consider the following code...

foreach (XElement e in GetGroups(DataList))
{
  if (e.Attribute("Name") == PageName || e.Attribute("Name") == "Generic")
  {
    PopulateList(e, ListToPopulate);
  }
}

The first part of the if statement should show as an error as you can't  apply the operator "==" to System.Xml.Linq.XAttribute and String.

Apparently.
-- 
Stuart

[Dennis :: Dr. Code]

Hello Stuart Hemming,

This is a known issue. JC currently doesn't check == operator types extensively.


All the best,
Dennis :: Dr. Code
the Telerik team

[Stuart Hemming]

Dear Dr Code,

Is that the equivalent of you telling me to take two aspirin and call back in the morning?

-- 
Stuart

[Dennis :: Dr. Code]

Hi Stuart,

Basically we'll fix the headache, but have to fix the heart disease first. Seriously, the bad-code-greens go with less priority than other issues (good-code-reds, exceptions, crashes). Just to let you know that your particular issue is on our list.


Sincerely yours,
Dennis :: Dr. Code
the Telerik team