#269 removed serialization

LDAPAccountManager · Feb 5, 2024 · a2e7e70 · a2e7e70
1 parent 92264a0
commit a2e7e70
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/lam/docs/manual-sources/chapter-installation.xml b/lam/docs/manual-sources/chapter-installation.xml
@@ -608,6 +608,14 @@
       version. Unless explicitly noticed there is no need to install an
       intermediate release.</para>
 
+      <section>
+        <title>8.6 -&gt; 8.7</title>
+
+        <para>LAM Pro: Self service profiles that were not saved for more than
+        3 years must be saved with LAM Pro 8.6 before upgrading to LAM Pro
+        8.7. </para>
+      </section>
+
       <section>
         <title>8.1 -&gt; 8.6</title>
 

diff --git a/lam/lib/selfService.inc b/lam/lib/selfService.inc
@@ -6,7 +6,7 @@ use function LAM\PERSISTENCE\dbTableExists;
 /*
 
   This code is part of LDAP Account Manager (http://www.ldap-account-manager.org/)
-  Copyright (C) 2006 - 2023  Roland Gruber
+  Copyright (C) 2006 - 2024  Roland Gruber
 
   This program is free software; you can redistribute it and/or modify
   it under the terms of the GNU General Public License as published by
@@ -476,12 +476,7 @@ class SelfServicePersistenceStrategyFileSystem implements SelfServicePersistence
 			if ($file) {
 				$data = fread($file, 10000000);
 				$profileData = @json_decode($data, true);
-				if ($profileData === null) {
-					$profile = unserialize($data);
-				}
-				else {
-					$profile = selfServiceProfile::import($profileData);
-				}
+				$profile = selfServiceProfile::import($profileData);
 				fclose($file);
 			}
 			else {