Merge pull request #34 from Planetbiru/feature/2.7.1

Feature/2.7.1

Author: kamshory

manual/includes/_entity.md

@@ -991,7 +991,7 @@ class SpecificationUtil
      * $@param array|null $additional
      * @return PicoSpecification
      */
-    public static function createSongSpecification($inputGet, $additional = null) //NOSONAR
+    public static function createSongSpecification($inputGet, $additional = null) // NOSONAR
     {
         $spesification = new PicoSpecification();
 
@@ -1212,7 +1212,7 @@ class SpecificationUtil
      * $@param array|null $additional
      * @return PicoSpecification
      */
-    public static function createSongDraftSpecification($inputGet, $additional = null) //NOSONAR
+    public static function createSongDraftSpecification($inputGet, $additional = null) // NOSONAR
     {
         $spesification = new PicoSpecification();
 

manual/includes/_secret-object.md

@@ -610,11 +610,11 @@ $cfg->loadYamlFile(dirname(__DIR__)."/.cfg/app.yml", true, true, true);
 $databaseCredentials = new PicoDatabaseCredentials($cfg->getDatabase());
 
 $database = new PicoDatabase($databaseCredentials, 
-    function($sql, $type) //NOSONAR
+    function($sql, $type) // NOSONAR
     {
         // callback when execute query that modify data
     }, 
-    function($sql) //NOSONAR
+    function($sql) // NOSONAR
     {
         // callback when execute all query
     }

manual/index.html

@@ -1305,11 +1305,11 @@ <h3>Secure Config from DynamicObject</h3>
 $databaseCredentials = new PicoDatabaseCredentials($cfg-&gt;getDatabase());
 
 $database = new PicoDatabase($databaseCredentials, 
-    function($sql, $type) //NOSONAR
+    function($sql, $type) // NOSONAR
     {
         // callback when execute query that modify data
     }, 
-    function($sql) //NOSONAR
+    function($sql) // NOSONAR
     {
         // callback when execute all query
     }
@@ -3254,7 +3254,7 @@ <h3>Find By Specification</h3>
      * $@param array|null $additional
      * @return PicoSpecification
      */
-    public static function createSongSpecification($inputGet, $additional = null) //NOSONAR
+    public static function createSongSpecification($inputGet, $additional = null) // NOSONAR
     {
         $spesification = new PicoSpecification();
 
@@ -3475,7 +3475,7 @@ <h3>Find By Specification</h3>
      * $@param array|null $additional
      * @return PicoSpecification
      */
-    public static function createSongDraftSpecification($inputGet, $additional = null) //NOSONAR
+    public static function createSongDraftSpecification($inputGet, $additional = null) // NOSONAR
     {
         $spesification = new PicoSpecification();
 

src/DataLabel/PicoDataLabels.php

@@ -41,7 +41,7 @@ public function append($data)
      */
     public function generate()
     {
-        foreach ($this->data as $data) //NOSONAR
+        foreach ($this->data as $data) // NOSONAR
         {
             // Implementation for processing each data label goes here
         }

src/DataTable.php

@@ -63,38 +63,54 @@ class DataTable extends SetterGetter
 
     /**
      * Current language code.
+     * 
+     * The property name starts with an underscore to prevent child classes 
+     * from overriding its value.
      *
      * @var string
      */
-    private $_currentLanguage; //NOSONAR
+    private $_currentLanguage; // NOSONAR
 
     /**
      * Language instances.
+     * 
+     * The property name starts with an underscore to prevent child classes 
+     * from overriding its value.
      *
      * @var PicoLanguage[]
      */
-    private $_lableLanguage = array(); //NOSONAR
+    private $_lableLanguage = array(); // NOSONAR
 
     /**
      * Table identity.
+     * 
+     * The property name starts with an underscore to prevent child classes 
+     * from overriding its value.
      *
      * @var PicoGenericObject
      */
-    private $_tableIdentity; //NOSONAR
+    private $_tableIdentity; // NOSONAR
 
     /**
      * Table information.
+     * 
+     * The property name starts with an underscore to prevent child classes 
+     * from overriding its value.
      *
      * @var PicoTableInfo
      */
-    private $_tableInfo; //NOSONAR
+    private $_tableInfo; // NOSONAR
 
     /**
      * Labels for the table.
+     * 
+     * The property name starts with an underscore to prevent child classes 
+     * from overriding its value.
      *
      * @var array
      */
-    private $_labels = array(); //NOSONAR
+    private $_labels = array(); // NOSONAR
+
 
     /**
      * Constructor

src/Database/PicoDatabase.php

@@ -37,7 +37,7 @@
  * @package MagicObject\Database
  * @link https://github.com/Planetbiru/MagicObject
  */
-class PicoDatabase //NOSONAR
+class PicoDatabase // NOSONAR
 {
     const QUERY_INSERT = "insert";
     const QUERY_UPDATE = "update";
@@ -735,22 +735,13 @@ public function fetchAll($sql, $tentativeType = PDO::FETCH_ASSOC, $defaultValue
      *
      * @param string $sql SQL query to be executed.
      * @param array|null $params Optional parameters to bind to the SQL query.
+     * @return PDOStatement|false Returns the PDOStatement object if successful, or `false` on failure.
      * @throws NullPointerException If the database connection is null.
+     * @throws PDOException If an error occurs while executing the query.
      */
     public function execute($sql, $params = null)
     {
-        if ($this->databaseConnection == null) {
-            throw new NullPointerException(self::DATABASE_NONECTION_IS_NULL);
-        }
-        
-        $this->executeDebug($sql, $params);
-        $stmt = $this->databaseConnection->prepare($sql);
-        
-        try {
-            $stmt->execute($params);
-        } catch (PDOException $e) {
-            // Handle exception as needed
-        }
+        return $this->executeQuery($sql, $params);
     }
 
     /**
@@ -898,8 +889,7 @@ private function executeDebug($query, $params = null)
             else
             {
                 call_user_func($this->callbackDebugQuery, $query);
-            }
-            
+            }           
         }
     }
 

src/Database/PicoDatabaseCredentials.php

@@ -53,14 +53,14 @@ class PicoDatabaseCredentials extends SecretObject
      * @DecryptOut
      * @var string
      */
-    protected $host = 'localhost';
+    protected $host;
 
     /**
      * Database server port.
      *
      * @var int
      */
-    protected $port = 3306;
+    protected $port;
 
     /**
      * Database username.
@@ -69,7 +69,7 @@ class PicoDatabaseCredentials extends SecretObject
      * @DecryptOut
      * @var string
      */
-    protected $username = "";
+    protected $username;
 
     /**
      * Database user password.
@@ -78,7 +78,7 @@ class PicoDatabaseCredentials extends SecretObject
      * @DecryptOut
      * @var string
      */
-    protected $password = "";
+    protected $password;
 
     /**
      * Database name.
@@ -87,7 +87,7 @@ class PicoDatabaseCredentials extends SecretObject
      * @DecryptOut
      * @var string
      */
-    protected $databaseName = "";
+    protected $databaseName;
 
     /**
      * Database schema (default: 'public').
@@ -96,14 +96,14 @@ class PicoDatabaseCredentials extends SecretObject
      * @DecryptOut
      * @var string
      */
-    protected $databaseSchema = "public"; 
+    protected $databaseSchema; 
 
     /**
      * Application time zone.
      *
      * @var string
      */
-    protected $timeZone = "Asia/Jakarta";
+    protected $timeZone;
 
     /**
      * Get the database driver.

src/Database/PicoPage.php

@@ -109,6 +109,21 @@ public function setPageSize($pageSize)
         $this->pageSize = max(1, intval($pageSize));
         return $this;
     }
+    
+    /**
+     * Calculates and retrieves the offset for database queries.
+     *
+     * The offset is used to determine the starting point for fetching data 
+     * in paginated queries, based on the current page number and page size.
+     *
+     * @return int The calculated offset for database queries.
+     */
+    public function getOffset()
+    {
+        $limit = $this->getPageSize();
+        $offset = ($this->getPageNumber() - 1) * $limit;
+        return max(0, $offset);
+    }
 
     /**
      * Calculates the limit and offset for database queries.