Skip to content

Conversation

@ZweiSteinSoft
Copy link
Contributor

Thanks @rlerdorf!

@ghost ghost assigned lem9 May 15, 2013
lem9 pushed a commit that referenced this pull request May 15, 2013
Fix most analysis results discovered in bug 3907
@lem9 lem9 merged commit 920d827 into phpmyadmin:QA_4_0 May 15, 2013
@lem9
Copy link
Contributor

lem9 commented May 15, 2013

Good job! I'm now fixing merge conflicts to master.

Apart from running hhvm, is there a way to know what remains to be fixed?

@rlerdorf
Copy link

I can run it again if you like. Let me know which branch to run it against.

@lem9
Copy link
Contributor

lem9 commented May 15, 2013

@rlerdorf you're very kind. Please run it on branches QA_4_0 and master.

@rlerdorf
Copy link

Here it is for QA_4_0 (only lightly filtered, so there could be false positives): https://gist.github.com/5585791
And for master: https://gist.github.com/5585809

@lem9
Copy link
Contributor

lem9 commented May 15, 2013

I have removed the dead code from QA_4_0 and master, which referred to unknown functions in replication.inc.php.

@lem9
Copy link
Contributor

lem9 commented May 16, 2013

Fixed:
libraries/rte/rte_export.lib.php
libraries/sql.lib.php
server_status_queries.php
libraries/Config.class.php

@ZweiSteinSoft ZweiSteinSoft deleted the bug-3907 branch May 21, 2013 18:35
@ZweiSteinSoft ZweiSteinSoft restored the bug-3907 branch May 21, 2013 18:36
@ZweiSteinSoft ZweiSteinSoft deleted the bug-3907 branch May 21, 2013 18:44
@ZweiSteinSoft
Copy link
Contributor Author

Fixed another one in e4199c1.

@ZweiSteinSoft
Copy link
Contributor Author

The other issues originate from TCPDF and php-gettext. @rlerdorf, we can't fix them ourselves. Did you get round to posting a report on Launchpad, for php-gettext?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants