Hi Chanaka,
I stumbled upon this piece of code (random pick) :
PMA_CommonFunctions::getInstance()->backquote($_REQUEST['view']['name'])
and it made me wonder if using a singleton for PMA_CommonFunctions is
necessary, because basicaly PMA_Commonfunctions is a collection of
methods, not really a 'living' object.
So then I had a look at the class, and I discovered that the class has
no class variables (apart from the _instance variable) and an empty
constructor.
So basicly this should be a static class, with static methods, because
no instance is needed for it to work.
And the above piece of code will become :
PMA_CommonFunctions::backquote($_REQUEST['view']['name'])
BTW: If you convert it to a static class, don't forget to replace the
$this->method() calls by the static equivalent self::method().
BTW2 : you forgot to add the if (! defined('PHPMYADMIN')), check in
the beginning of the file, see [0].
[0] http://wiki.phpmyadmin.net/pma/File_template
--
Kind regards,
Dieter Adriaenssens
See <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/1863/changes>
Changes:
[weblate] Translated using Weblate.
[weblate] Translated using Weblate.
[weblate] Translated using Weblate.
[madhura.cj] bug #3561224 [edit] cell edit date field with empty date fills in current date
------------------------------------------
Started by GitHub push by
Building in workspace <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/>
Checkout:workspace / <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/> - hudson.remoting.LocalChannel@582d0ceb
Using strategy: Default
Last Built Revision: Revision 2069f77e571519251a5fec79f91522856dae2dd9 (origin/master)
Fetching changes from 1 remote Git repository
Fetching upstream changes from git://github.com/phpmyadmin/phpmyadmin.git
Commencing build of Revision 9bb9d62b951d38c7e4122eceda733935319b265b (origin/master)
Checking out Revision 9bb9d62b951d38c7e4122eceda733935319b265b (origin/master)
FATAL: Unable to produce a script file
hudson.util.IOException2: Failed to create a temp file on <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/>
at hudson.FilePath.createTextTempFile(FilePath.java:1090)
at hudson.tasks.CommandInterpreter.createScriptFile(CommandInterpreter.java:104)
at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:66)
at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:58)
at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:19)
at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:717)
at hudson.model.Build$BuildExecution.build(Build.java:199)
at hudson.model.Build$BuildExecution.doRun(Build.java:160)
at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:499)
at hudson.model.Run.execute(Run.java:1502)
at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:46)
at hudson.model.ResourceController.execute(ResourceController.java:88)
at hudson.model.Executor.run(Executor.java:236)
Caused by: java.io.IOException: No space left on device
at java.io.FileOutputStream.writeBytes(Native Method)
at java.io.FileOutputStream.write(FileOutputStream.java:297)
at sun.nio.cs.StreamEncoder.writeBytes(StreamEncoder.java:220)
at sun.nio.cs.StreamEncoder.implClose(StreamEncoder.java:315)
at sun.nio.cs.StreamEncoder.close(StreamEncoder.java:148)
at java.io.OutputStreamWriter.close(OutputStreamWriter.java:233)
at hudson.FilePath$14.invoke(FilePath.java:1084)
at hudson.FilePath$14.invoke(FilePath.java:1068)
at hudson.FilePath.act(FilePath.java:842)
at hudson.FilePath.act(FilePath.java:824)
at hudson.FilePath.createTextTempFile(FilePath.java:1068)
... 12 more
Build step 'Execute shell' marked build as failure
[CHECKSTYLE] Skipping publisher since build result is FAILURE
[DRY] Collecting duplicate code analysis files...
[DRY] Finding all files that match the pattern build/logs/pmd-cpd.xml
[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 19 warnings.
[DRY] Computing warning deltas based on reference build #1862
[TASKS] Skipping publisher since build result is FAILURE
[ANALYSIS-COLLECTOR] Computing warning deltas based on reference build #1862
Recording plot data
[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] - Test reports were found but not all of them are new. Did all the tests run?
* <http://ci.phpmyadmin.net/job/phpMyAdmin-continuous/ws/build/logs/junit.xml> is 1 day 20 hr old
[xUnit] [INFO] - Setting the build status to FAILURE
[xUnit] [INFO] - Stopping recording.
Hi
please try to merge every change done in QA_3_5 into master so that
future changes can be merged easier. Today I was unable to do merge
because these commits (second one is just a fix for first one) do not
apply at all.
In case the changes should not be merge at all, just do:
git checkout master
git merge -s ours origin/QA_3_5
This will tell git that it should not care about extra commits in
QA_3_5 anymore.
So master is now missing following commits and I have no clue if there
is something needed there or not:
commit a483935cf8aac81a2d87390af36f387100f3bdf2
Author: J.M <me(a)mynetx.net>
Date: Wed Aug 22 11:14:45 2012 +0200
bug #3559925 [privileges] Incorrect updating of the list of users
commit e0383e6b1938dd5a5fd377e27d04c3880b6c2d6d
Author: Marc Delisle <marc(a)infomarc.info>
Date: Wed Aug 22 06:35:26 2012 -0400
Remove extraneous characters
--
Michal Čihař | http://cihar.com | http://blog.cihar.com
Hi,
I'm getting familiar with tbl_chart.* to port the feature to jqplot.
This "Query results operations>Display chart" feature was a GSoC 2010
project and is documented at [0].
I just noticed that tbl_chart.php fetches the whole data set generated
by the query and puts it into an array. Then, this is passed to the js
level for chart generation. Evidently, the web server's process and the
browser itself hit memory limits, even for medium tables
(sakila.film_actor).
I'm not sure what the user expects when clicking on Display chart in
terms of the number of rows, because, for example "Query results
operations>Print view" only displays the current rows on screen (30 by
default).
So I suggest to do something similar to what was done in Search>Zoom
search: to have in the chart interface a control for the maximum number
of rows, with 500 as the default value.
Comments?
[0] http://wiki.phpmyadmin.net/pma/Charts
--
Marc Delisle
http://infomarc.info
This is a desperate request for help. I'm fighting the issue for 2 days!!
I installed TomCat on Ubuntu (an Amazon instance) and since
then lost my phpmyadmin access to my DB.
I keep on getting:
"Forbidden
You don't have permission to access /phpmyadmin on this server."
I googled the subject to death and checked, virtually, everything against
another Amazon instance were things are fine.
I'll pay for help!
Thanks in advance!
-Michael
Hi all,
I've found some obviously broken HTML tags on tbl_structure.php and I
assume that this is due to the recent refactoring of this page. I
couldn't find where the issue is in the code, but it manifests itself in
the "more" dropdown as links there are now not clickable. Screenshot
attached.
Another thing that I noticed in the new *_structure code are the new
function names. Not sure if it's just me, but after a 2 minute look I
found all of these functions where IMO the names can be improved:
PMA_getHtmlForPrintViewAndDataDictionaryLinks - too long?
PMA_SortableTableHeader - starts with a capital unlike others
PMA_getAliasAndTruename - incorrect camel case
PMA_getStuffForEnginetable - meaningless + incorrect camel case
PMA_getHtmlForCheckAlltableColumn - incorrect camel case
PMA_getHtmlForSomeLinks - meaningless
PMA_getHtmlForRowStatstableRow - incorrect camel case
getHtmlForRowStatsTable - no PMA_ prefix
PMA_getHtmlForDistincValueAction - spelling: "Distinct"
PMA_getHtmlForActionsIntableStructure - incorrect camel case
Bye,
Rouslan
Hi Marc,
I'm looking [1] bug in the table operation page,
I 'git checkout' to very old commit and check the same bug in there, But
still this bug was there,
I found that, first bad commit was 4d2fcd910bc7e07114fcc76f55705c1a6e8c685e
(This is not a my commit)
I can't figure out where is this error happen.
[1] -
http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23067&atid…
--
Regards.
Thilina Abeyrathna
Gtalk : thilinaabeyrathna
skype: thilinabuddika88
thilinaa.wordpress.com