comparison core/lib/Drupal/Core/Session/WriteSafeSessionHandler.php @ 5:12f9dff5fda9 tip

Update to Drupal core 8.7.1

 <?php
 
 namespace Drupal\Core\Session;
 
 /**
- * Wraps another SessionHandlerInterface to prevent writes when not allowed.
+ * Wraps the session handler to prevent writes when not necessary or allowed.
  */
 class WriteSafeSessionHandler implements \SessionHandlerInterface, WriteSafeSessionHandlerInterface {
 
   /**
    * @var \SessionHandlerInterface

    * Whether or not the session is enabled for writing.
    *
    * @var bool
    */
   protected $sessionWritable;
+
+  /**
+   * The read sessions.
+   *
+   * @var array
+   *   Session data keyed by the session ID.
+   */
+  private $readSessions;
 
   /**
    * Constructs a new write safe session handler.
    *
    * @param \SessionHandlerInterface $wrapped_session_handler

 
   /**
    * {@inheritdoc}
    */
   public function read($session_id) {
-    return $this->wrappedSessionHandler->read($session_id);
+    $value = $this->wrappedSessionHandler->read($session_id);
+    $this->readSessions[$session_id] = $value;
+    return $value;
   }
 
   /**
    * {@inheritdoc}
    */
   public function write($session_id, $session_data) {
+    // Only write the session when it has been modified.
+    if (isset($this->readSessions[$session_id]) && $this->readSessions[$session_id] === $session_data) {
+      return TRUE;
+    }
     if ($this->isSessionWritable()) {
       return $this->wrappedSessionHandler->write($session_id, $session_data);
     }
-    else {
-      return TRUE;
-    }
+    return TRUE;
   }
 
   /**
    * {@inheritdoc}
    */