Patch #25731

Use only one if with two conditions in issue show instead of two ifs (if in if)

Added by Marius BALTEANU 23 days ago. Updated 23 days ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:


Category:Code cleanup/refactoring
Target version:3.4.0


I think that there are no reasons to use two ifs in source:trunk/app/views/issues/show.html.erb#L67. I propose to use only one with both conditions.

use_only_one_if.patch Magnifier (870 Bytes) Marius BALTEANU, 2017-05-01 16:39


#1 Updated by Go MAEDA 23 days ago

  • Target version set to 3.4.0

Thanks for the patch. Setting target version to 3.4.0.

Also available in: Atom PDF