Changeset 3407972

Timestamp:

12/02/2025 12:22:11 PM (3 months ago)

Author:

galdub

Message:

trunk 1.3.8

Location:

poptin/trunk

Files:

• poptin.php (modified) (4 diffs)
• readme.txt (modified) (2 diffs)

poptin/trunk/poptin.php

@@ -4 +4 @@
 Contributors: poptin, galdub, tomeraharon
 Description: Use Poptin to get more leads, sales, and email subscribers. Create targeted beautiful pop ups and forms in less than 2 minutes with ease.
-Version: 1.3.7
+Version: 1.3.8
 Author: Poptin
 Author URI: https://www.poptin.com
@@ -173 +173 @@
         if(isset($_GET['page']) && $_GET['page'] == 'Poptin-support') {
             wp_redirect(admin_url("admin.php?page=poptin"));
+            exit;
+        }
+        
+        // Handle logout redirect early, before any output is sent
+        if(isset($_GET['page']) && $_GET['page'] == 'poptin-logout') {
+            // Check user permissions
+            if (!current_user_can('manage_options')) {
+                wp_die(__('You do not have sufficient permissions to access this page.', 'ppbase'));
+            }
+            
+            // Clear Poptin data
+            update_option('poptin_id', '');
+            update_option('poptin_marketplace_token', '');
+            update_option('poptin_marketplace_email_id', '');
+            update_option('poptin_user_id', '');
+            poptin_clear_all_caches();
+            
+            // Redirect to Poptin dashboard
+            wp_redirect(admin_url('admin.php?page=poptin'));
             exit;
         }
@@ -832 +851 @@
 
     public function poptin_logout_view() {
-        // Clear Poptin data (similar to your existing delete_poptin_id method)
+        // This is a fallback - the logout should be handled in admin_init
+        // But if we reach here, clear data and use JavaScript redirect
         update_option('poptin_id', '');
         update_option('poptin_marketplace_token', '');
@@ -839 +859 @@
         poptin_clear_all_caches();
         
-        // Redirect back to main page
-        wp_redirect(admin_url('admin.php?page=poptin'));
-        exit;
+        // Use JavaScript redirect since headers may already be sent
+        $redirect_url = admin_url('admin.php?page=poptin');
+        ?>
+        <script type="text/javascript">
+            window.location.href = "<?php echo esc_js($redirect_url); ?>";
+        </script>
+        <p><?php esc_html_e('Logging out... If you are not redirected automatically, ', 'ppbase'); ?><a href="<?php echo esc_url($redirect_url); ?>"><?php esc_html_e('click here', 'ppbase'); ?></a>.</p>
+        <?php
     }
 

poptin/trunk/readme.txt

@@ -4 +4 @@
 Requires at least: 3.1
 Tested up to: 6.9
-Stable tag: 1.3.7
+Stable tag: 1.3.8
 Plugin URI: https://www.poptin.com
 License: GPLv3
@@ -243 +243 @@
 == Changelog ==
 
+= 1.3.8 =
+Logout bug fixed
+
 = 1.3.7 =
 WordPress Playground bug fixed