Changeset 3452434

Timestamp:

02/02/2026 10:29:02 PM (2 weeks ago)

Author:

shift8

Message:

Fixed minor UI bug

Location:

shift8-real-estate-listings-for-treb/trunk

Files:

• admin/class-shift8-treb-admin.php (modified) (1 diff)
• includes/class-shift8-treb-sync-service.php (modified) (2 diffs)
• readme.txt (modified) (2 diffs)
• shift8-treb.php (modified) (3 diffs)

shift8-real-estate-listings-for-treb/trunk/admin/class-shift8-treb-admin.php

@@ -397 +397 @@
             // Get the main plugin instance and trigger sync
             $plugin = Shift8_TREB::get_instance();
-            $plugin->sync_listings_cron();
+            $plugin->sync_listings_cron(true);
             
             shift8_treb_log('=== MANUAL SYNC COMPLETED ===', array(), 'info');

shift8-real-estate-listings-for-treb/trunk/includes/class-shift8-treb-sync-service.php

@@ -164 +164 @@
             }
 
+            $current_timestamp = current_time('c'); // ISO 8601 format
+
             if (empty($listings)) {
+                if (!$dry_run) {
+                    update_option('shift8_treb_last_sync', $current_timestamp);
+                }
+
                 $results['message'] = 'No listings returned from API';
                 shift8_treb_log('No listings returned from AMPRE API', array(
                     'settings' => array(
                         'listing_age_days' => $this->settings['listing_age_days'] ?? 30,
-                        'last_sync_timestamp' => $this->settings['last_sync_timestamp'] ?? 'none'
+                        'last_sync_timestamp' => $this->settings['last_sync_timestamp'] ?? 'none',
+                        'last_sync_updated' => !$dry_run
                     )
                 ));
@@ -282 +289 @@
             // Update last sync timestamp (only if not dry run)
             if (!$dry_run) {
-                $current_timestamp = current_time('c'); // ISO 8601 format
                 update_option('shift8_treb_last_sync', $current_timestamp);
             }

shift8-real-estate-listings-for-treb/trunk/readme.txt

@@ -5 +5 @@
 Tested up to: 6.8
 Requires PHP: 7.4
-Stable tag: 1.7.3
+Stable tag: 1.7.4
 License: GPLv3 or later
 License URI: https://www.gnu.org/licenses/gpl-3.0.html
@@ -132 +132 @@
 
 == Changelog ==
+
+= 1.7.4 =
+* Fix: "Last sync" timestamp now updates even when API returns zero listings (manual/cron)
+* Tests: added coverage to ensure last sync updates on empty results and not on dry-run
 
 = 1.7.3 =

shift8-real-estate-listings-for-treb/trunk/shift8-treb.php

@@ -4 +4 @@
  * Plugin URI: https://github.com/stardothosting/shift8-treb
  * Description: Integrates Toronto Real Estate Board (TREB) listings via AMPRE API, automatically importing property listings into WordPress. Replaces the Python script with native WordPress functionality.
- * Version: 1.7.3
+ * Version: 1.7.4
  * Author: Shift8 Web
  * Author URI: https://shift8web.ca
@@ -22 +22 @@
 
 // Plugin constants
-define('SHIFT8_TREB_VERSION', '1.7.3');
+define('SHIFT8_TREB_VERSION', '1.7.4');
 define('SHIFT8_TREB_PLUGIN_FILE', __FILE__);
 define('SHIFT8_TREB_PLUGIN_DIR', plugin_dir_path(__FILE__));
@@ -513 +513 @@
      * @since 1.0.0
      */
-    public function sync_listings_cron() {
+    public function sync_listings_cron($skip_lock = false) {
         // Check if sync is already running (prevent simultaneous syncs)
-        $sync_lock = get_transient('shift8_treb_sync_lock');
-        if ($sync_lock) {
-            shift8_treb_log('Sync already running, skipping cron execution', array(
-                'lock_time' => $sync_lock,
-                'current_time' => time()
-            ));
-            return;
+        if (!$skip_lock) {
+            $sync_lock = get_transient('shift8_treb_sync_lock');
+            if ($sync_lock) {
+                shift8_treb_log('Sync already running, skipping cron execution', array(
+                    'lock_time' => $sync_lock,
+                    'current_time' => time()
+                ));
+                return;
+            }
         }