Improve test behaviour
This commit is contained in:
parent
271b6cb8f9
commit
69db696ab1
3 changed files with 26 additions and 17 deletions
|
@ -955,6 +955,8 @@ class dba {
|
||||||
* @return boolean Was the command executed successfully?
|
* @return boolean Was the command executed successfully?
|
||||||
*/
|
*/
|
||||||
public static function rollback() {
|
public static function rollback() {
|
||||||
|
$ret = false;
|
||||||
|
|
||||||
switch (self::$driver) {
|
switch (self::$driver) {
|
||||||
case 'pdo':
|
case 'pdo':
|
||||||
if (!self::$db->inTransaction()) {
|
if (!self::$db->inTransaction()) {
|
||||||
|
|
|
@ -412,11 +412,17 @@ class App
|
||||||
public function set_baseurl($url)
|
public function set_baseurl($url)
|
||||||
{
|
{
|
||||||
$parsed = @parse_url($url);
|
$parsed = @parse_url($url);
|
||||||
|
$hostname = '';
|
||||||
|
|
||||||
if (x($parsed)) {
|
if (x($parsed)) {
|
||||||
|
if (!empty($parsed['scheme'])) {
|
||||||
$this->scheme = $parsed['scheme'];
|
$this->scheme = $parsed['scheme'];
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!empty($parsed['host'])) {
|
||||||
$hostname = $parsed['host'];
|
$hostname = $parsed['host'];
|
||||||
|
}
|
||||||
|
|
||||||
if (x($parsed, 'port')) {
|
if (x($parsed, 'port')) {
|
||||||
$hostname .= ':' . $parsed['port'];
|
$hostname .= ':' . $parsed['port'];
|
||||||
}
|
}
|
||||||
|
|
|
@ -781,12 +781,10 @@ class Image
|
||||||
$img_str = Network::fetchUrl($url, true, $redirects, 4);
|
$img_str = Network::fetchUrl($url, true, $redirects, 4);
|
||||||
$filesize = strlen($img_str);
|
$filesize = strlen($img_str);
|
||||||
|
|
||||||
// The tests are failing with an read error. This can be caused by memory shortage
|
try {
|
||||||
// See https://stackoverflow.com/questions/10175758/getimagesize-read-error
|
if (function_exists("getimagesizefromstring")) {
|
||||||
// So we use the alternate method instead
|
$data = getimagesizefromstring($img_str);
|
||||||
//if (function_exists("getimagesizefromstring")) {
|
} else {
|
||||||
// $data = getimagesizefromstring($img_str);
|
|
||||||
//} else {
|
|
||||||
$tempfile = tempnam(get_temppath(), "cache");
|
$tempfile = tempnam(get_temppath(), "cache");
|
||||||
|
|
||||||
$a = get_app();
|
$a = get_app();
|
||||||
|
@ -796,7 +794,10 @@ class Image
|
||||||
|
|
||||||
$data = getimagesize($tempfile);
|
$data = getimagesize($tempfile);
|
||||||
unlink($tempfile);
|
unlink($tempfile);
|
||||||
//}
|
}
|
||||||
|
} catch (Exception $e) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
if ($data) {
|
if ($data) {
|
||||||
$data["size"] = $filesize;
|
$data["size"] = $filesize;
|
||||||
|
|
Loading…
Reference in a new issue