<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Hi,</div><div><br></div></div><div class="gmail_quote">On 29 October 2015 at 10:56, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Madhura and Atul,<br>
<br>
I was looking at indexAction() in DatabaseStructureController.class.php,<br>
which is 566 lines. I find it too long to be manageable, and PMD agrees [0].<br>
<br>
Do you consider this method a work in progress? Does MVC mandate that<br>
indexAction() should remain that size?<br>
<br></blockquote><div> <br></div><div></div></div><div class="gmail_extra">I agree it's a work in progress and needs further refactoring. Looking at it, I think it pretty much contains only the logic that just displays the Database_Structure index page (or may be the logic for "Drops/deletes/etc. multiple tables if required" needs to go into separate action?). Thereafter, it can surely be further broken into some helper functions or sub-actions for readability/reuseability.</div><div class="gmail_extra">Also, it's size will further reduce once other actions like addRemoveFavoriteTablesAction are called directly, instead of, via indexAction.</div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Regards</div><div><br></div><div>Atul</div></div></div>
</div></div>