June 2012 - Developers - lists.phpmyadmin.net

[Phpmyadmin-devel] Automated Testing - GSoC 2012
by yasitha pandithawatta 25 Jun '12

25 Jun '12

Hi Michal, Though I started to writing test cases for common library I came to know Chanaka is re-factoring it. So it is worthless of writing tests for current code. So I moved in to core library. I want to know any re-factoring going on core library. One more thing. Whats the best way to update our forked repository? Using git pull or git fetch and merge it? Or doesn't have much difference? Regards, -- Yasitha Pandithawatta Undergraduate Computer Science and Engineering Department University of Moratuwa Sri Lanka

2 3

[Phpmyadmin-devel] Refactoring db_search.php
by Atul Pratap Singh 19 Jun '12

19 Jun '12

Hi Marc, While taking a look at the Db-search functionality I noticed a bug. Steps: 1. Open db-search tab for a database. 2. Make a search by entering a keyword and selecting all tables. 3. From the 'browse' and 'delete' links in search results, select 'delete' for any table. 4. Notice that a DELETE query is sent to sql.php. But no actual deletion happens. Regards -- Atul Pratap Singh Senior Undergraduate Student Institute Of Technology, Banaras Hindu University, Varanasi, India.

3 12

[Phpmyadmin-devel] AjaxEnable directive
by Rouslan Placella 19 Jun '12

19 Jun '12

Hi Dieter, as I just wrote in my blog post, I'm considering dropping the $cfg['AjaxEnable'] directive as in my opinion it will considerably complicate the processing of some pages and doesn't seem to any real benefits. Your thoughts on this appreciated. Bye, Rouslan

4 8

[Phpmyadmin-devel] Writing useful docblocks
by Michal Čihař 19 Jun '12

19 Jun '12

Hi all, especially GSoC students just a quick reminder, that docblocks are meant to document function and it's parameters, so writing docblocks as following is quite useless: /** * * @param type $srid * @param type $scale_data * @param type $output */ - You should describe what the function is supposed to do - You should document actual types of parameters instead of writing there "type" - You should document parameter meaning Thank you for your cooperation in making readable and easy to understand code. -- Michal Čihař | http://cihar.com | http://blog.cihar.com

2 1

[Phpmyadmin-devel] GSoC - Removal of frames
by Dieter Adriaenssens 19 Jun '12

19 Jun '12

Hi Rouslan, I went through your latest commits in your GSoC branch, see github for the small remarks. You did a great job on creating the different classes (PMA_Header, PMA_Footer, PMA_Response, ...) moving functionality from other library files into it, and integrating these new classes in the current code base. I think you've come to a point where these changes are ready to be merged with master. What do you think? -- Kind regards, Dieter Adriaenssens

4 5

[Phpmyadmin-devel] Merging GSoC branches
by Michal Čihař 18 Jun '12

18 Jun '12

Hi all I've just merged two GSoC branches, one just adds few testcases what should be safe, the second brings partial rewrite of table browsing code, what might be more problematic, but I think the code is quite safe to merge. -- Michal Čihař | http://cihar.com | http://phpmyadmin.cz

1 0

[Phpmyadmin-devel] character "1" appears at top and bottom of pages in master
by Isaac Bennetch 18 Jun '12

18 Jun '12

Hi, In the current master -- "Git revision: 0954bfb from master branch, committed on Jun 16, 2012 at 01:02 AM by Marc Delisle", I see the character 1 at the top of both the navigation frame, near the top of the main frame, and also at the bottom of the main frame. I'm away from my main system at the moment, so I'm afraid I can't 'git-bisect' to the problem commit, sorry. Regards

3 4

[Phpmyadmin-devel] Refactoring edit: bug with transformations
by Marc Delisle 18 Jun '12

18 Jun '12

Hi Thilina, when using grid-editing (which calls tbl_replace.php), normally I see the generated UPDATE statement after my edit. However, if my table uses a transformation (for example, a VARCHAR with a MIME type = text/plain and a Browser transformation = text/plain: substr), I don't see the generated UPDATE statement. As a hint to debug this, I saw that PMA_getTransformationFunctionAndTransformationOptions() does not (always?) return an array, or an array with is correct to feed to $response->addJSON(). -- Marc Delisle http://infomarc.info

2 1

[Phpmyadmin-devel] Refactoring tbl_replace_fields.inc.php
by Thilina Buddika Abeyrathna 18 Jun '12

18 Jun '12

Hi Marc. Above file is not a huge file. And it includes only in tbl_replace.php script. (if I wrong, please correct me) So we can implement above file as a function in insert_edit.lib.php file. Is it ok? (The bug you mentioned yesterday not fixed yet. I'm looking at that also) -- Regards. Thilina Buddika Abeyrathna, Department of Computer Engineering, Faculty Of Engineering, University of Peradeniya, Sri Lanka.

2 5

[Phpmyadmin-devel] Build failed in Jenkins: phpMyAdmin-continuous #1559
by michal＠cihar.com 17 Jun '12

17 Jun '12

See <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/1559/changes> Changes: [rouslan] Return string from PMA_userprefs_autoload_header() [rouslan] Renamed PMA_userprefs_autoload_header() to PMA_userprefsAutoloadGetHeader() [rouslan] Return string from PMA_reloadNavigation() [rouslan] Renamed PMA_reloadNavigation() to PMA_getReloadNavigationScript() [rouslan] Return string from PMA_addRecentTable() [rouslan] New classes PMA_Header and PMA_Scripts [rouslan] Dropped header.inc.php in favor of the PMA_Header class [rouslan] Removed some references to the non-existent header.inc.php file [rouslan] Use PMA_Header class in themes.php [rouslan] The PMA_Menu class no longer need to be a singleton [rouslan] Use PMA_Header class in browse_foreigners.php [rouslan] Drop constant PMA_DISPLAY_HEADING [rouslan] Got rid of some dead code from gis_data_editor.php [rouslan] Drop old header_*.inc.php scripts from querywindow.php [rouslan] Use PMA_Header class in navigation.php [rouslan] PMA_Header::$headerIsSent must be static [rouslan] Integrated print view functionality into the PMA_Header class [rouslan] Dropped header_http.inc.php [rouslan] Small code cleanup [rouslan] Fixed invalid method invocation [rouslan] Integrated pmd with the new Header class [rouslan] Integrated authentication libraries with PMA_Header class [rouslan] Removed the last references to the old header files [rouslan] Fixed JS bug in user prefs [rouslan] Docblocks and coding style fixes for the new classes [rouslan] Added PMA_Footer and PMA_Footnotes classes [rouslan] Handle scripts in the PMA_Footer class [rouslan] Moved the rendering of debug messages to the PMA_Footer class [rouslan] Moved the rendering of error messages to the PMA_Footer class [rouslan] Fixed rendering when there are no footnotes [rouslan] Coding style fixes [rouslan] Moved PMA_setHistory call to PMA_Footer class [rouslan] Added PMA_Response class [rouslan] Integrated chk_rel.php with PMA_Response class [rouslan] PMA_Footer and PMA_Header no longer need to be singletons [rouslan] Added PMA_OutputBuffering class, dropped ob.lib.php [rouslan] relation.lib.php is already included by common.inc.php [rouslan] Fixes for disabling the response [rouslan] Integrated themes.php with PMA_Response class [rouslan] Finished integrating the PMA_Footer class [rouslan] Don't force 'text/html' on ajax requests [rouslan] Fixed multiple errors display [rouslan] Added JSON responses to the PMA_Response class [rouslan] Added missing library include [rouslan] Dropped footer.inc.php [rouslan] Fixed undefined variable [rouslan] Don't display error messages in ajax responses [rouslan] Integrated ajax responses from user_password.php with PMA_Response class [rouslan] Fix typo [rouslan] improve DocBlock [rouslan] Fixed variable name [rouslan] PHP seems to reset the CWD after a call to exit, so we need to collect the file timestamps early [rouslan] Dropped PMA_includeJS() in favor of the PMA_Scripts class [rouslan] The filename and line number sometimes are unavailable [rouslan] Display "Too many error messages" only once [rouslan] Disable warnings in header for cookie auth [rouslan] Drop display() from Header and Footer classes. The Response class will do all the rendering. [rouslan] Remember CWD after the initial script has shutdown [rouslan] Fixed bad variable name [rouslan] Ensure that PMA_Response is instanciated in common.inc.php [rouslan] Disable Response class for XHR requests in PMD [rouslan] Don't die before the Response class is instanciated [rouslan] Dropped $GLOBALS['page_title'] [rouslan] BBCode is never used with PMA_showHint() in the code base [rouslan] Completely drop footnotes and use tooltips instead [rouslan] Docblock fixes [rouslan] Display messages once on password change [rouslan] Force JSON response for AJAX requests [rouslan] Fixed ajax response for change password form [rouslan] Fixed ajax response for "display git revision" [rouslan] Moved the check for whether warnings are enabled inside function [rouslan] Use PMA_Response class in PMA_ajaxResponse() [rouslan] Expect JSON for every ajax request in JavaScript [rouslan] Fixed conflicting global variable names [rouslan] Fixed user preferences export/import [rouslan] Fixed ajax responses in server_variable.php [rouslan] Dropped PMA_ajaxResponse() [rouslan] Added missing API call to the PMA_Footer class [rouslan] Coding style fixes [rouslan] Moved lost global [rouslan] Fixed config auth [rouslan] Set headerIsSent property of PMA_Header class as private, ------------------------------------------ Started by an SCM change Building in workspace <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/> Checkout:workspace / <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/> - hudson.remoting.LocalChannel@5999ef99 Using strategy: Default Last Built Revision: Revision bf219df66dab3a7190d63f6226bbd7de33348956 (origin/master) Fetching changes from 1 remote Git repository Fetching upstream changes from git://github.com/phpmyadmin/phpmyadmin.git Commencing build of Revision dc8f8829194859837b0b32488691cd3dd3258a77 (origin/master) Checking out Revision dc8f8829194859837b0b32488691cd3dd3258a77 (origin/master) [workspace] $ /bin/sh -xe /tmp/hudson8843342657442270387.sh + ./scripts/generate-mo --quiet [workspace] $ ant clean phpunit phpcpd phpcs phploc Unable to locate tools.jar. Expected to find it in /usr/lib/jvm/java-6-openjdk-amd64/lib/tools.jar Buildfile: <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build.xml> clean: [delete] Deleting directory <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/api> [delete] Deleting directory <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/code-browser> [delete] Deleting directory <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/coverage> [delete] Deleting directory <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/logs> [delete] Deleting directory <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/pdepend> [mkdir] Created dir: <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/api> [mkdir] Created dir: <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/code-browser> [mkdir] Created dir: <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/coverage> [mkdir] Created dir: <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/logs> [mkdir] Created dir: <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/pdepend> phpunit: [exec] PHPUnit 3.6.10 by Sebastian Bergmann. [exec] [exec] Configuration read from <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/phpunit.xml.dist> [exec] [exec] ..........................................FI........II......... 63 / 772 ( 8%) [exec] ............................................................... 126 / 772 ( 16%) [exec] .............EEI..................F............................ 189 / 772 ( 24%) [exec] .............................................EPHP Fatal error: Class 'PMA_Response' not found in <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/libraries/error.inc.p…> on line 17 [exec] PHP Stack trace: [exec] PHP 1. {main}() /usr/bin/phpunit:0 [exec] PHP 2. PHPUnit_TextUI_Command::main() /usr/bin/phpunit:46 [exec] PHP 3. PHPUnit_TextUI_Command->run() /usr/share/php/PHPUnit/TextUI/Command.php:130 [exec] PHP 4. PHPUnit_TextUI_TestRunner->doRun() /usr/share/php/PHPUnit/TextUI/Command.php:192 [exec] PHP 5. PHPUnit_Framework_TestSuite->run() /usr/share/php/PHPUnit/TextUI/TestRunner.php:325 [exec] PHP 6. PHPUnit_Framework_TestSuite->run() /usr/share/php/PHPUnit/Framework/TestSuite.php:705 [exec] PHP 7. PHPUnit_Framework_TestSuite->run() /usr/share/php/PHPUnit/Framework/TestSuite.php:705 [exec] PHP 8. PHPUnit_Framework_TestSuite->runTest() /usr/share/php/PHPUnit/Framework/TestSuite.php:745 [exec] PHP 9. PHPUnit_Framework_TestCase->run() /usr/share/php/PHPUnit/Framework/TestSuite.php:772 [exec] PHP 10. PHPUnit_Framework_TestResult->run() /usr/share/php/PHPUnit/Framework/TestCase.php:751 [exec] PHP 11. PHP_Invoker->invoke() /usr/share/php/PHPUnit/Framework/TestResult.php:647 [exec] PHP 12. call_user_func_array() /usr/share/php/PHP/Invoker.php:93 [exec] PHP 13. PHPUnit_Framework_TestCase->runBare() /usr/share/php/PHP/Invoker.php:93 [exec] PHP 14. PHPUnit_Framework_TestCase->runTest() /usr/share/php/PHPUnit/Framework/TestCase.php:804 [exec] PHP 15. ReflectionMethod->invokeArgs() /usr/share/php/PHPUnit/Framework/TestCase.php:942 [exec] PHP 16. PMA_fatalError_test->testFatalErrorMessageWithArgs() /usr/share/php/PHPUnit/Framework/TestCase.php:942 [exec] PHP 17. PMA_fatalError() <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/test/libraries/core/P…>:36 [exec] PHP 18. include() <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/libraries/core.lib.php>:234 [exec] Result: 255 phpcpd: [exec] phpcpd 1.3.5 by Sebastian Bergmann. [exec] [exec] Found 11 exact clones with 348 duplicated lines in 7 files: [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:628-654 [exec] /libraries/schema/Pdf_Relation_Schema.class.php:650-676 [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:282-300 [exec] /libraries/schema/Svg_Relation_Schema.class.php:277-295 [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:424-440 [exec] /libraries/schema/Svg_Relation_Schema.class.php:391-407 [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:472-480 [exec] /libraries/schema/Svg_Relation_Schema.class.php:441-449 [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:607-657 [exec] /libraries/schema/Svg_Relation_Schema.class.php:588-638 [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:424-440 [exec] /libraries/schema/Visio_Relation_Schema.class.php:222-238 [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:472-480 [exec] /libraries/schema/Visio_Relation_Schema.class.php:272-280 [exec] [exec] - /libraries/schema/Eps_Relation_Schema.class.php:607-663 [exec] /libraries/schema/Visio_Relation_Schema.class.php:375-431 [exec] [exec] - /libraries/schema/Dia_Relation_Schema.class.php:752-773 [exec] /libraries/schema/Visio_Relation_Schema.class.php:577-598 [exec] [exec] - /themes/original/css/common.css.php:1271-1292 [exec] /themes/original/css/common.css.php:1275-1296 [exec] [exec] - /themes/original/css/common.css.php:1215-1323 [exec] /themes/pmahomme/css/common.css.php:1574-1682 [exec] [exec] 0.28% duplicated lines out of 125745 total lines of code. [exec] [exec] Time: 31 seconds, Memory: 103.25Mb [exec] Result: 1 phpcs: [exec] Time: 02:08, Memory: 177.25Mb [exec] [exec] Result: 1 phploc: [exec] phploc 1.6.4 by Sebastian Bergmann. [exec] [exec] Directories: 24 [exec] Files: 324 [exec] [exec] Lines of Code (LOC): 125745 [exec] Cyclomatic Complexity / Lines of Code: 0.15 [exec] Comment Lines of Code (CLOC): 37319 [exec] Non-Comment Lines of Code (NCLOC): 88426 [exec] [exec] Namespaces: 0 [exec] Interfaces: 0 [exec] Classes: 87 [exec] Abstract: 2 (2.30%) [exec] Concrete: 85 (97.70%) [exec] Average Class Length (NCLOC): 216 [exec] Methods: 841 [exec] Scope: [exec] Non-Static: 742 (88.23%) [exec] Static: 99 (11.77%) [exec] Visibility: [exec] Public: 674 (80.14%) [exec] Non-Public: 167 (19.86%) [exec] Average Method Length (NCLOC): 22 [exec] Cyclomatic Complexity / Number of Methods: 3.64 [exec] [exec] Anonymous Functions: 0 [exec] Functions: 854 [exec] [exec] Constants: 119 [exec] Global constants: 103 [exec] Class constants: 16 BUILD SUCCESSFUL Total time: 3 minutes 1 second [CHECKSTYLE] Collecting checkstyle analysis files... [CHECKSTYLE] Parsing 1 files in <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/> [CHECKSTYLE] Successfully parsed file <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/logs/checkstyle…> of module with 7392 warnings. [CHECKSTYLE] Computing warning deltas based on reference build #1558 [DRY] Collecting duplicate code analysis files... [DRY] Parsing 1 files in <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/> [DRY] Successfully parsed file <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/logs/pmd-cpd.xml> of module with 20 warnings. [DRY] Computing warning deltas based on reference build #1558 [TASKS] Scanning folder '<http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/'> for files matching the pattern '**/*.php' - excludes: build/**,libraries/PHPExcel/**,libraries/tcpdf/**,libraries/php-gettext/** [TASKS] Found 431 files to scan for tasks [TASKS] Found 185 open tasks. [TASKS] Computing warning deltas based on reference build #1558 [ANALYSIS-COLLECTOR] Computing warning deltas based on reference build #1558 Recording plot data Publishing Clover coverage report... Clover xml file does not exist in: <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/logs> called: clover.xml and will not be copied to: /var/lib/jenkins/jobs/phpMyAdmin-continuous/builds/2012-06-13_20-31-20/clover.xml Could not find 'build/logs/clover.xml'. Did you generate the XML report for Clover? [htmlpublisher] Archiving HTML reports... [htmlpublisher] Archiving at PROJECT level <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/coverage> to /var/lib/jenkins/jobs/phpMyAdmin-continuous/htmlreports/Code_Coverage ERROR: Directory '<http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/coverage'> exists but failed copying to '/var/lib/jenkins/jobs/phpMyAdmin-continuous/htmlreports/Code_Coverage'. ERROR: This is especially strange since your build otherwise succeeded. Build step 'Publish HTML reports' changed build result to FAILURE [xUnit] [INFO] - Starting to record. [xUnit] [INFO] - Processing PHPUnit-3.4 (default) [xUnit] [INFO] - [PHPUnit-3.4 (default)] - 1 test report file(s) were found with the pattern 'build/logs/junit.xml' relative to '<http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/'> for the testing framework 'PHPUnit-3.4 (default)'. [xUnit] [ERROR] - The result file '<http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/logs/junit.xml'> for the metric 'PHPUnit' is empty. The result file has been skipped. [xUnit] [INFO] - Setting the build status to FAILURE [xUnit] [INFO] - Stopping recording.

1 7