<br><br><div class="gmail_quote">On Mon, Aug 1, 2011 at 7:03 PM, Michal Čihař <span dir="ltr"><<a href="mailto:michal@cihar.com">michal@cihar.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi<br>
<br>
Dne Fri, 22 Jul 2011 22:34:55 +0530<br>
Madhura Jayaratne <<a href="mailto:madhura.cj@gmail.com">madhura.cj@gmail.com</a>> napsal(a):<br>
<div class="im"><br>
> Looking at the CheckStyle reports from the CI, all the export files show<br>
> very high counts of indentation rule violations.<br>
> The code in the 'else' block of 'if (isset($plugin_list))' condition is<br>
> wrongly indented in all of these files.<br>
><br>
> There are two possible ways to correct this.<br>
> 1) Correctly indent the 'else' block<br>
> 2) Add 'return' statement to the end of 'if' block and remove 'else {' part.<br>
><br>
> Which one if preferred?<br>
<br>
</div>I don't think 2 will work, so out of these 1 is only solution. Rewrite<br>
it so that same functions are not defined in all plugins (using classes<br>
instead) would be preferable solution, though it is much more complex<br>
than reindenting :-).<br>
<font color="#888888"><br></font></blockquote><div> </div><div>Yep, I was comparing it with the import plugins and didn't notice that these contains methods.</div><div><span class="Apple-style-span">So what is preferred, </span>reindenting <span class="Apple-style-span">as a quick solution or letting it there as it is till it is rewritten in OOP?</span></div>
<div><br></div></div>-- <br>Thanks and Regards,<div><br></div><div>Madhura Jayaratne<br><div><br></div></div><br>