Skip to content

Commit

Permalink
Automatic code style fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
annda committed Jan 25, 2024
1 parent 9a026de commit a7a40fb
Show file tree
Hide file tree
Showing 8 changed files with 64 additions and 64 deletions.
7 changes: 2 additions & 5 deletions Functions.php
Original file line number Diff line number Diff line change
@@ -1,18 +1,17 @@
<?php

/**
* @noinspection PhpUndefinedMethodInspection
* @noinspection PhpComposerExtensionStubsInspection
*/


namespace dokuwiki\plugin\sqlite;

/**
* SQLite registered functions
*/
class Functions
{

/**
* Register all standard functions
*
Expand Down Expand Up @@ -74,7 +73,7 @@ public static function GroupConcatFinalize(&$context, $rownumber)
if (empty($context['data'][0])) {
return null;
}
return join($context['sep'], $context['data']);
return implode($context['sep'], $context['data']);
}


Expand Down Expand Up @@ -119,7 +118,6 @@ public static function pageExists($pageid)
static $cache = [];
if (!isset($cache[$pageid])) {
$cache[$pageid] = page_exists($pageid);

}
return (int)$cache[$pageid];
}
Expand Down Expand Up @@ -157,5 +155,4 @@ public static function resolvePage($page, $context)
resolve_pageid($ns, $page, $exists);
return $page;
}

}
1 change: 0 additions & 1 deletion QuerySaver.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@

class QuerySaver
{

protected $db;
protected $upstream;

Expand Down
20 changes: 10 additions & 10 deletions SQLiteDB.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
*/
class SQLiteDB
{
const FILE_EXTENSION = '.sqlite3';
public const FILE_EXTENSION = '.sqlite3';

/** @var \PDO */
protected $pdo;
Expand Down Expand Up @@ -239,8 +239,10 @@ public function saveRecord($table, $data, $replace = true)
}

/** @noinspection SqlResolve */
$sql = $command . ' INTO "' . $table . '" (' . join(',', $columns) . ') VALUES (' . join(',',
$placeholders) . ')';
$sql = $command . ' INTO "' . $table . '" (' . implode(',', $columns) . ') VALUES (' . implode(
',',
$placeholders
) . ')';
$stm = $this->query($sql, $values);
$success = $stm->rowCount();
$stm->closeCursor();
Expand Down Expand Up @@ -371,7 +373,7 @@ public function dumpToFile($filename)
$sql = "SELECT * FROM " . $table['name'];
$res = $this->query($sql);
while ($row = $res->fetch(\PDO::FETCH_ASSOC)) {
$values = join(',', array_map(function ($value) {
$values = implode(',', array_map(function ($value) {
if ($value === null) return 'NULL';
return $this->pdo->quote($value);
}, $row));
Expand Down Expand Up @@ -415,7 +417,7 @@ protected function applyMigrations()
'sqlite' => $this->helper,
'adapter' => $this,
];
$event = new \Doku_Event('PLUGIN_SQLITE_DATABASE_UPGRADE', $data);
$event = new Event('PLUGIN_SQLITE_DATABASE_UPGRADE', $data);

$this->pdo->beginTransaction();
try {
Expand All @@ -425,11 +427,9 @@ protected function applyMigrations()
foreach ($sql as $query) {
$this->pdo->exec($query);
}
} else {
if (!$event->result) {
// advise before returned false, but the result was false
throw new \PDOException('Plugin event did not signal success');
}
} elseif (!$event->result) {
// advise before returned false, but the result was false
throw new \PDOException('Plugin event did not signal success');
}
$this->setOpt('dbversion', $newVersion);
$this->pdo->commit();
Expand Down
35 changes: 18 additions & 17 deletions Tools.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@

namespace dokuwiki\plugin\sqlite;

class Tools {

class Tools
{
/**
* Split sql queries on semicolons, unless when semicolons are quoted
*
Expand All @@ -13,36 +13,37 @@ class Tools {
* @param string $sql
* @return string[] sql queries
*/
public static function SQLstring2array($sql) {
$statements = array();
public static function SQLstring2array($sql)
{
$statements = [];
$len = strlen($sql);

// Simple state machine to "parse" sql into single statements
$in_str = false;
$in_com = false;
$statement = '';
for($i=0; $i<$len; $i++){
$prev = $i ? $sql[$i-1] : "\n";
for ($i = 0; $i < $len; $i++) {
$prev = $i ? $sql[$i - 1] : "\n";
$char = $sql[$i];
$next = $i < ($len - 1) ? $sql[$i+1] : '';
$next = $i < ($len - 1) ? $sql[$i + 1] : '';

// in comment? ignore everything until line end
if($in_com){
if($char == "\n"){
if ($in_com) {
if ($char == "\n") {
$in_com = false;
}
continue;
}

// handle strings
if($in_str){
if($char == "'"){
if($next == "'"){
if ($in_str) {
if ($char == "'") {
if ($next == "'") {
// current char is an escape for the next
$statement .= $char . $next;
$i++;
continue;
}else{
} else {
// end of string
$statement .= $char;
$in_str = false;
Expand All @@ -55,20 +56,20 @@ public static function SQLstring2array($sql) {
}

// new comment?
if($char == '-' && $next == '-' && $prev == "\n"){
if ($char == '-' && $next == '-' && $prev == "\n") {
$in_com = true;
continue;
}

// new string?
if($char == "'"){
if ($char == "'") {
$in_str = true;
$statement .= $char;
continue;
}

// the real delimiter
if($char == ';'){
if ($char == ';') {
$statements[] = trim($statement);
$statement = '';
continue;
Expand All @@ -77,7 +78,7 @@ public static function SQLstring2array($sql) {
// some standard query stuff
$statement .= $char;
}
if($statement) $statements[] = trim($statement);
if ($statement) $statements[] = trim($statement);

return array_filter($statements); // remove empty statements
}
Expand Down
17 changes: 7 additions & 10 deletions admin.php
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
<?php

use dokuwiki\Extension\AdminPlugin;
use dokuwiki\Form\Form;
use dokuwiki\Form\InputElement;
use dokuwiki\plugin\sqlite\QuerySaver;
Expand All @@ -12,13 +13,13 @@
* @license GPL 2 http://www.gnu.org/licenses/gpl-2.0.html
* @author Andreas Gohr <[email protected]>
*/
class admin_plugin_sqlite extends DokuWiki_Admin_Plugin
class admin_plugin_sqlite extends AdminPlugin
{
/** @var SQLiteDB */
protected $db = null;
protected $db;

/** @var QuerySaver */
protected $querySaver = null;
protected $querySaver;

/** @inheritdoc */
public function getMenuSort()
Expand Down Expand Up @@ -148,12 +149,7 @@ public function getTOC()
$dbfiles = glob($conf['metadir'] . '/*.sqlite3');
if (is_array($dbfiles)) foreach ($dbfiles as $file) {
$db = basename($file, '.sqlite3');
$toc[] = array(
'link' => wl($ID, array('do' => 'admin', 'page' => 'sqlite', 'db' => $db, 'sectok' => getSecurityToken())),
'title' => $db,
'level' => 2,
'type' => 'ul',
);
$toc[] = ['link' => wl($ID, ['do' => 'admin', 'page' => 'sqlite', 'db' => $db, 'sectok' => getSecurityToken()]), 'title' => $db, 'level' => 2, 'type' => 'ul'];
}

return $toc;
Expand Down Expand Up @@ -238,7 +234,8 @@ protected function selfLink($form = true, $params = [])
'page' => 'sqlite',
'db' => $this->db ? $this->db->getDBName() : '',
'sectok' => getSecurityToken(),
], $params
],
$params
);

return wl($ID, $params, false, $form ? '&' : '&amp;');
Expand Down
42 changes: 23 additions & 19 deletions helper.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,22 +6,24 @@
* @noinspection PhpComposerExtensionStubsInspection
*/

use dokuwiki\Extension\Plugin;
use dokuwiki\plugin\sqlite\SQLiteDB;
use dokuwiki\plugin\sqlite\Tools;



/**
* For compatibility with previous adapter implementation.
*/
if(!defined('DOKU_EXT_PDO')) define('DOKU_EXT_PDO', 'pdo');
if (!defined('DOKU_EXT_PDO')) define('DOKU_EXT_PDO', 'pdo');
class helper_plugin_sqlite_adapter_dummy
{
public function getName() {
public function getName()
{
return DOKU_EXT_PDO;
}

public function setUseNativeAlter($set) {}
public function setUseNativeAlter($set)
{
}
}

/**
Expand All @@ -31,10 +33,10 @@ public function setUseNativeAlter($set) {}
* @author Andreas Gohr <[email protected]>
* @deprecated 2023-03-15
*/
class helper_plugin_sqlite extends DokuWiki_Plugin
class helper_plugin_sqlite extends Plugin
{
/** @var SQLiteDB|null */
protected $adapter = null;
protected $adapter;

/** @var array result cache */
protected $data;
Expand Down Expand Up @@ -89,7 +91,7 @@ public function existsPDOSqlite()
*/
public function init($dbname, $updatedir)
{
if(!defined('DOKU_UNITTEST')) { // for now we don't want to trigger the deprecation warning in the tests
if (!defined('DOKU_UNITTEST')) { // for now we don't want to trigger the deprecation warning in the tests
dbg_deprecated(SQLiteDB::class);
}

Expand All @@ -108,7 +110,7 @@ public function init($dbname, $updatedir)
* @param SQLiteDB $adapter
* @return void
*/
function setAdapter($adapter)
public function setAdapter($adapter)
{
$this->adapter = $adapter;
}
Expand Down Expand Up @@ -182,17 +184,18 @@ public function query()
* @return bool|string
* @throws Exception
*/
public function prepareSql($args) {
public function prepareSql($args)
{

$sql = trim(array_shift($args));
$sql = rtrim($sql, ';');

if(!$sql) {
if (!$sql) {
throw new \Exception('No SQL statement given', -1);
}

$argc = count($args);
if($argc > 0 && is_array($args[0])) {
if ($argc > 0 && is_array($args[0])) {
$args = $args[0];
$argc = count($args);
}
Expand All @@ -201,18 +204,18 @@ public function prepareSql($args) {
$qmc = substr_count($sql, '?');
if ($argc < $qmc) {
throw new \Exception('Not enough arguments passed for statement. ' .
'Expected '.$qmc.' got '. $argc.' - '.hsc($sql));
} elseif($argc > $qmc) {
'Expected ' . $qmc . ' got ' . $argc . ' - ' . hsc($sql));
} elseif ($argc > $qmc) {
throw new \Exception('Too much arguments passed for statement. ' .
'Expected '.$qmc.' got '. $argc.' - '.hsc($sql));
'Expected ' . $qmc . ' got ' . $argc . ' - ' . hsc($sql));
}

// explode at wildcard, then join again
$parts = explode('?', $sql, $argc + 1);
$args = array_map([$this->adapter->getPdo(), 'quote'], $args);
$sql = '';

while(($part = array_shift($parts)) !== null) {
while (($part = array_shift($parts)) !== null) {
$sql .= $part;
$sql .= array_shift($args);
}
Expand Down Expand Up @@ -356,7 +359,7 @@ public function countChanges($res)
public function quote_and_join($vals, $sep = ',')
{
$vals = array_map([$this->adapter->getPdo(), 'quote'], $vals);
return join($sep, $vals);
return implode($sep, $vals);
}

/**
Expand Down Expand Up @@ -391,7 +394,7 @@ public function escape_string($str)
*/
public function SQLstring2array($sql)
{
if(!DOKU_UNITTEST) { // for now we don't want to trigger the deprecation warning in the tests
if (!DOKU_UNITTEST) { // for now we don't want to trigger the deprecation warning in the tests
dbg_deprecated(Tools::class . '::SQLstring2array');
}
return Tools::SQLstring2array($sql);
Expand All @@ -400,7 +403,8 @@ public function SQLstring2array($sql)
/**
* @deprecated needs to be fixed in stuct and structpublish
*/
public function doTransaction($sql, $sqlpreparing = true) {
public function doTransaction($sql, $sqlpreparing = true)
{
throw new \Exception(
'This method seems to never have done what it suggests. Please use the query() function instead.'
);
Expand Down
Loading

0 comments on commit a7a40fb

Please sign in to comment.