| Reporter |
Oskar Gewalli (Oskar.Gewalli)
|
|---|---|
| Created | Oct 31, 2011 12:44:40 PM |
| Updated | Feb 13, 2012 8:26:36 PM |
| Resolved | Nov 11, 2011 7:36:14 PM |
| Priority | Normal |
| Type | Bug |
| Fix versions | No Fix versions |
| State | Won't fix |
| Assignee | Evgeny Pasynkov (pasynkov) |
| Subsystem | Code Analysis |
| Affected versions | 6.0 |
| Fixed in build | No Fixed in build |
Reports parameter 'item' is only used for precondition checks:
private void CheckBusinessId(Item item)
{
if (item.BusinessId != _session.CurrentBusinessId)
{
throw new BusinessIdMissmatchException();
}
}
This is essential business logic. This must run at all times and is not merely a simple assertion.
private void CheckBusinessId(Item item)
{
if (item.BusinessId != _session.CurrentBusinessId)
{
throw new BusinessIdMissmatchException();
}
}
This is essential business logic. This must run at all times and is not merely a simple assertion.