<br><br><div class="gmail_quote">On Sun, May 13, 2012 at 5:37 PM, Madhura Jayaratne <span dir="ltr"><<a href="mailto:madhura.cj@gmail.com" target="_blank">madhura.cj@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br><br><div class="gmail_quote">On Sun, May 13, 2012 at 12:42 PM, Thilina Buddika Abeyrathna <span dir="ltr"><<a href="mailto:thilinaabeyrathna@gmail.com" target="_blank">thilinaabeyrathna@gmail.com</a>></span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Marc,<div>when I'm writing functions for columns of insertForm I need to pass all the following parameters to the functions.</div>




<div><div>$field, $backup_field, $field_name_appendix, $unnullify_trigger, $tabindex, $tabindex_for_value, $idindex, $data</div>

<div>So function signature is very long which is somewhat ugly, Is it ok to have a global parameter array for insert_edit.lib.php file. Does this break the refactoring?</div><span><font color="#888888"><div>
<br></div></font></span></div></blockquote></div></div>Hi Thilina,<br><br>Whenever you encounter a long list of parameters, I think it is a good idea to evaluate whether your method is doing too much inside it, whether it is possible to logically break your function into few smaller ones, each requiring only few parameters.<div>



Secondly if the parameters logically fit together you can introduce parameter objects that would encapsulate the parameters as Poitr pointed out in some other thread.</div><div>If non of the above applies I agree with Marc as it would make things clearer.</div>

<span class="HOEnZb"><font color="#888888">
<div><div><br></div>-- <br>Thanks and Regards,<div><br></div><div>Madhura Jayaratne<br>
<div><br></div></div></div></font></span></blockquote><div><br></div><div>Hi Marc,</div><div>There are HTML code block from line 438 in the tbl_change.php file(after my last commit [0]).  It doesn't mix with much PHP tags. It is somewhat readable.</div>

<div>Do I need to implement several functions to render this HTML block. Can you give me a suggestion..</div><div> </div></div><div>[0] - <a href="https://github.com/thilinaa/phpmyadmin/commit/9d6c38daf6a98d915a34391bddb987b116be98c0">https://github.com/thilinaa/phpmyadmin/commit/9d6c38daf6a98d915a34391bddb987b116be98c0</a></div>

-- <br><font face="georgia, serif">Regards.</font><br><br><font face="'trebuchet ms', sans-serif" size="4">Thilina Buddika Abeyrathna,</font><br><font face="georgia, serif">Department of Computer Engineering,</font><br>

<font face="georgia, serif">Faculty Of Engineering,</font><br><font face="georgia, serif">University of Peradeniya,</font><div><font face="georgia, serif">Sri Lanka.<br><br></font></div><br>