This will 2 things:

1) The first change prevents "expensive" code in HTML::toBBcode() to be executed
   just for an empty string which makes no sense.
2) The above change was maybe flawed as $apcontact['about'] would have never
   been created, not even empty which could have side effects

Thanks to @annando to make me rethink this part of code.

diff --git a/src/Model/APContact.php b/src/Model/APContact.php
index e99a692a183f341081bfd99192a59f2ccbfe265d..69f9be5135095d3df00c0eb35dcc58073dc94c50 100644 (file)
--- a/src/Model/APContact.php
+++ b/src/Model/APContact.php
@@ -242,9 +242,7 @@ class APContact
                }
 
                $aboutHtml = JsonLD::fetchElement($compacted, 'as:summary', '@value');
-               if ($aboutHtml != "") {
-                       $apcontact['about'] = HTML::toBBCode($aboutHtml);
-               }
+               $apcontact['about'] = (!empty($aboutHtml) ? HTML::toBBCode($aboutHtml) : '');
 
                $ims = JsonLD::fetchElementArray($compacted, 'vcard:hasInstantMessage');