<div dir="ltr">Hi all,<div><br></div><div>I tried to refrain from asking help as much as I can, but now I am almost done with this feature request and I figure I need to re-factor a large portion of the code and it would be safe if I ascertain the purpose of some specific chunks of the code written so that I don't break anything while re-factoring. </div>
<div><br></div><div>I sincerely wish the people reading this mail not to think of this asking for help mail as my inability to resolve problems by myself.</div><div><br></div><div>This is the farthest I got <a href="http://adityasastry.in/multicolumn.jpg">http://adityasastry.in/multicolumn.jpg</a> (Screenshot functionality is not working in Ubuntu for some reason, so I had to manually take a photo from my phone of the screen and upload it).</div>
<div><br></div><div>As you can see the arrows are not pointing in the correct direction also the url's attached to those arrows don't work yet. For these problems to resolve I figure I had to re-factor getOrderLinkAndSortedHeaderHtml() function. This function accepts columns one at a time and checks with the isInSorted() function to ascertain if it is a sorted column in that case it stops the execution there itself and constructs the target url. I have rewritten the isInSorted() function and getSortingUrlParams() function to work with multiple columns and now I need to refactor this function to work with multiple columns.</div>
<div><br></div><div>While refactoring I had a couple of doubts.</div><div><br></div><div>(A) What is the difference between $fields_meta->orgname and $fields_meta->name ?</div><div><br></div><div>(B) What purpose does this chunk of code in getOrderLinkAndSortedHeaderHtml() between lines 1900 and 1920 serve ?</div>
<div><br></div><div><div> if (preg_match(</div><div>            '@(.*)([[:space:]](LIMIT (.*)|PROCEDURE (.*)|FOR UPDATE|'</div><div>            . 'LOCK IN SHARE MODE))@is',</div><div>            $unsorted_sql_query, $regs3</div>
<div>        )) {</div><div>            $sorted_sql_query = $regs3[1] . $sort_order . $regs3[2];</div><div>        } else {</div><div>            $sorted_sql_query = $unsorted_sql_query . $sort_order;</div><div>        }</div>
</div><div><br></div><div>(C) There is some direction parameter used in most of the code in this file which takes two values horizontal and vertical.</div><div>What does this parameter stand for ? What semantic sense do these parameters horizontal and vertical make ?</div>
<div><br></div><div>$direction == self::DISP_DIR_HORIZONTAL_FLIPPED<br></div><div><br>Regards,<div>Aditya</div>
</div></div>