16 Aug
2008
16 Aug
'08
3:18 p.m.
Crack a écrit :
A few changes to PMA code
patch at http://trac.nq.pl/pma/browser/trunk/patches/pma_dev.patch
==================================================================
most changes are obvious
I changed only_db to an array as internally it makes no diffrence to PMA
and it's easier for me to handle variables that have only one type. I
think the only
place where this change matters is _checkOnlyDatabase() in
List_Database.class.php (fix also in patch)
Hi Piotr,
Is this change for only_db needed by your GSoC code? Because I don't see
your point. The "if" just before this one transforms the string into an
array if there is something inside it. And we use strlen() instead of
empty() to support the case where the db name is '0'.
Other typos and comment fixes merged in trunk.
import.lib.php
==============
PMA_importRunQuery()
> unused $timeout_passed in global
> $cfg mentioned two times in global
Merged.
export.php
==========
$onfly_compression = $GLOBALS['cfg']['CompressOnFly'] &&
($compression
== 'gzip' | $compression == 'bzip');
$onfly_compression = $GLOBALS['cfg']['CompressOnFly'] &&
($compression
== 'gzip' || $compression == 'bzip');
> use the logic OR operator, there's no need for binary operations here
Merged.
> I can't find where "on the fly"
compression is enabled, is it used at all?
The CompressOnFly comes from config.inc.php.
tbl_properties.inc.php
======================
$hint = '<p>' .
sprintf($strMIME_transformation_note,
'<a href="transformation_overview.php?'
. PMA_generate_common_url($db, $table) . '"
target="_blank">',
'</a>') . '</p>';
> <p> isn't allowed inside of <li>
We are not inside a <li> at this point.
Config.class.php
================
PMA_Config::getThemeUniqueValue()
reading pma_fontsize cookie may produce a notice when it isn't set,
add isset check
Ok, added by precaution.
Piotr