Hi All,
While I'm refactoring some files relating to display query results (currently display_tbl.lib.php, common.lib.php), I can method names are used regardless of the inside functionality.
Ex :- "PMA_displayHtmlRadio" in common.lib.php method return a string of HTML content.
But the name tells the function displays HTML itself.
As I feel these things should be modified.
As library files, they never should render HTML inside the functions themselves.
Better thing is to return a string of PHP having the HTML content.
So my suggestion is to use the phrase "get" for the every method which return any data type.
So that developer can understand it returns something just looking at the name.
Following are some suggested names. (Some of them are already discussed in this mailing list)
Ex:
PMA_displayHtmlRadio -> PMA_getRadioFields
PMA_generateSliderEffect -> PMA_getDivForSliderEffect
PMA_generateHtmlDropdown -> PMA_getDropdown
PMA_displayTableNavigationButton -> getTableNavigationButton
(All above functions return strings)
Hope to join a discussion on this.
Regards !
--
____________________________________
Chanaka Indrajith
Bsc.Computer Engineering Undergraduate
Faculty of Engineering
University of Peradeniya
Sri Lanka
____________________________________