Merge pull request #48 from sectsect/feature/refactor-june2024

refactor: code refactor

README.md

@@ -95,25 +95,21 @@ add_filter( 'google_ss2db_before_save', function ( $row, $worksheetid, $workshee
 
 And also use `add_filter('google_ss2db_after_save', $return_array )` to perform any processing with the return value.
 ```php
-add_filter( 'google_ss2db_after_save', function ( $array ) {
-  if ( 'My Spreadsheet' === $worksheetname ) {
-    // $id              = $array['id'];
-    // $date            = $array['date'];
-    // $title           = $array['title'];
-    // $value           = $array['value'];
-    // $work_sheet_id   = $array['worksheet_id'];
-    // $work_sheet_name = $array['worksheet_name'];
-    // $sheet_name      = $array['sheet_name'];
-    // $result          = $array['result'];
-
-    // Do something...
-    $return = $something;
-  } else {
-    $return = $array;
+add_filter( 'google_ss2db_after_save', function ( $data ) {
+  if ( 'My Spreadsheet' === $data['worksheet_name'] ) {
+    // $id              = $data['id'];
+    // $date            = $data['date'];
+    // $title           = $data['title'];
+    // $value           = $data['value'];
+    // $work_sheet_id   = $data['worksheet_id'];
+    // $work_sheet_name = $data['worksheet_name'];
+    // $sheet_name      = $data['sheet_name'];
+    // $result          = $data['result'];
+
+    // Example
+    my_callback( $data );
   }
-
-  return $return;
-} );
+});
 ```
 
 ## APIs

includes/save.php

@@ -176,9 +176,10 @@ function save_spreadsheet(): array {
 	);
 }
 
-$return  = save_spreadsheet();
-$rus     = apply_filters( 'google_ss2db_after_save', $return );
-$bool    = ( $return['result'] ) ? true : false;
+$data = save_spreadsheet();
+apply_filters( 'google_ss2db_after_save', $data );
+
+$bool    = (bool) $data['result'];
 $referer = wp_unslash( $_POST['_wp_http_referer'] );
 $referer = str_replace( '&settings-updated=true', '', $referer );
 $referer = $referer . '&ss2dbupdated=' . $bool;