On Tue, May 8, 2012 at 7:23 PM, Chanaka Dharmarathna <pe.chanaka.ck@gmail.com> wrote:


On Tue, May 8, 2012 at 7:08 PM, Marc Delisle <marc@infomarc.info> wrote:
Le 2012-05-08 09:25, Chanaka Dharmarathna a écrit :

>>
>> See "Class Variables and Methods" in [0].
>>
>> [0] http://pear.php.net/manual/en/standards.naming.php

> Hi Marc,
>
> I went through that document.
> If the function is global, it will be like 'PMA_getDivForSliderEffect()'
> And if it is a local function (for particular class), it will be like
> 'getDivForSliderEffect()'.

If this is a public method, yes. If a private one, prefix its name with
an underscore.

>
> Though I went through that document before this, I couldn't get that point.
> Thanks for pointing out that.



--
Marc Delisle
http://infomarc.info


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Phpmyadmin-devel mailing list
Phpmyadmin-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel


Got it Marc.
Thanks.


--
____________________________________

Chanaka Indrajith
Bsc.Computer Engineering Undergraduate
Faculty of Engineering
University of Peradeniya
Sri Lanka
____________________________________


Hi Michal,

I just did my first commit to my repository [0].
In that commit I have done some refactoring on "PMA_displayTableHeaders()" function in "display_tbl.php" file.
All HTML renderings by "echo"s were removed and those HTML was rendered at the bottom of the function at once.
As well some related functions in "common.lib.php" file also were modified in order to return a string of HTML content.
But the tests were not yet modified according to my modifications.
Since there was a conflict after my first local commit [1] (git commit), I had to resolve conflicts and commit again. (that is [0])

I have already installed phpUnit framework.
It will be very helpful if you can suggest me some documentation to setup and run tests in PMA (selenium etc).

I'm intending to do the above refactoring procedure to all the functions in "display_tbl.lib.php" file before moving to any other task.

[0] : https://github.com/Chanaka/phpmyadmin/commit/dfc73d3494445ff9430e5d0b4f33f800c6e480e0
[1] : https://github.com/Chanaka/phpmyadmin/commit/004f4f5258a0820975e2da9fa02a6d3ba6622368

Regards !
--
____________________________________

Chanaka Indrajith
Bsc.Computer Engineering Undergraduate
Faculty of Engineering
University of Peradeniya
Sri Lanka
____________________________________