Completed
Pull Request — master (#2282)
by ྅༻ Ǭɀħ
01:46
created

includes/ezSQL/ez_sql_mysql_yourls.php (3 issues)

Check that methods always have a scope defined

Best Practice Minor

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

1
<?php
2
3 View Code Duplication
class ezSQL_mysql_YOURLS extends ezSQL_mysql {
4
5
	/**
6
	 * Return MySQL server version
7
	 *
8
	 * @since 1.7
9
	 */
10
	function mysql_version() {
0 ignored issues
show
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
11
		return mysql_get_server_info( $this->dbh ) ;
12
	}
13
	
14
    /**
15
     * Comply to YOURLS debug mode
16
     *
17
     * @since 1.7.1
18
     */
19
    function __construct( $user, $pass, $name, $host ) {
0 ignored issues
show
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
20
        $this->show_errors = defined( 'YOURLS_DEBUG' ) && YOURLS_DEBUG;
21
        parent::__construct( $user, $pass, $name, $host );
22
    }
23
	
24
	/**
25
	 * Perform mySQL query
26
	 *
27
	 * Added to the original function: logging of all queries
28
	 *
29
	 * @since 1.7
30
	 */
31
	function query( $query ) {
0 ignored issues
show
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
32
	
33
		// Keep history of all queries
34
		$this->debug_log[] = $query;
35
36
		// Original function
37
		return parent::query( $query );
38
	}
39
	
40
}
41
42