feat(invitations) Added token generation, storage and check

We will need to get the token from GET most likely.

issue #5850

invitations/admin/invitations/class_invitation.inc

@@ -140,7 +140,9 @@ class invitation extends simplePlugin
       if (empty($email)) {
         continue;
       }
-      if (mail_utf8($email, FALSE, $this->from_mail, $this->fdInvitationEmailSubject, $this->fdInvitationEmailContent)) {
+      $token  = standAlonePage::generateRandomHash();
+      $body   = sprintf($this->fdInvitationEmailContent, $token);
+      if (mail_utf8($email, FALSE, $this->from_mail, $this->fdInvitationEmailSubject, $body)) {
         $tabObject  = objects::create('registration');
         $baseObject = $tabObject->getBaseObject();
 
@@ -149,6 +151,7 @@ class invitation extends simplePlugin
         $baseObject->fdRegistrationState        = 'sent';
         $baseObject->fdRegistrationLastChange   = date('c').':'.$ui->dn;
         $baseObject->fdRegistrationInvitationDN = $this->dn;
+        $baseObject->fdRegistrationToken        = $token;
 
         $messages = $tabObject->save();
         if (!empty($messages)) {

invitations/admin/invitations/class_registration.inc

@@ -69,6 +69,7 @@ class registration extends simplePlugin
             _('User object'), _('User object created by this registration'),
             'fdRegistrationUserDN', FALSE
           ),
+          new HiddenAttribute('fdRegistrationToken'),
         )
       ),
     );

invitations/contrib/openldap/invitations-fd.schema

@@ -69,6 +69,12 @@ attributetype ( 1.3.6.1.4.1.38414.69.11.5 NAME 'fdRegistrationUserDN'
   SYNTAX 1.3.6.1.4.1.1466.115.121.1.15
   SINGLE-VALUE )
 
+attributetype ( 1.3.6.1.4.1.38414.69.11.6 NAME 'fdRegistrationToken'
+  DESC 'FusionDirectory - Token to identify this registration'
+  EQUALITY octetStringMatch
+  SYNTAX 1.3.6.1.4.1.1466.115.121.1.40{128}
+  SINGLE-VALUE )
+
 # Objectclasses
 
 objectclass ( 1.3.6.1.4.1.38414.69.2.1 NAME 'fdInvitation'

public-forms/include/class_publicFormPage.inc

@@ -23,6 +23,8 @@ class publicFormPage extends standAlonePage
   protected $form;
   protected $template;
   protected $done;
+  // TODO fill this var by link/GET/LLNG
+  protected $registrationToken;
 
   function readLdapConfig()
   {
@@ -33,6 +35,8 @@ class publicFormPage extends standAlonePage
 
   function save_object()
   {
+    global $ui;
+
     if (!$this->activated) {
       return;
     }
@@ -77,10 +81,24 @@ class publicFormPage extends standAlonePage
     $this->template->save_object();
 
     if (isset($_POST['form_submit'])) {
-      $tabobject  = $this->template->apply();
-      $errors     = $tabobject->save();
+      $userTabObject  = $this->template->apply();
+      $errors         = $userTabObject->save();
       if (empty($errors)) {
         $this->done = TRUE;
+        if (isset($this->registrationToken)) {
+          $registrations = objects::ls('registration', NULL, NULL, '(fdRegistrationToken='.$this->registrationToken.')');
+          if (count($registrations) == 1) {
+            $tabObject  = objects::open(key($registrations), 'registration');
+            $baseObject = $tabObject->getBaseObject();
+
+            $baseObject->fdRegistrationUserDN     = $userTabObject->dn;
+            $baseObject->fdRegistrationState      = 'filled';
+            $baseObject->fdRegistrationLastChange = date('c').':'.$ui->dn;
+
+            $messages = $tabObject->save();
+            msg_dialog::displayChecks($messages);
+          }
+        }
       } else {
         msg_dialog::displayChecks($errors);
         // TODO - Find a way to refill previous values in the fields

[Ghost User]

Complete the task associated to this "TODO" comment.

[Ghost User]

SonarQube analysis reported 2 issues

• 1 minor
• 1 info

Watch the comments in this conversation to review them.

1 extra issue

Note: The following issues were found on lines that were not modified in the commit. Because these issues can't be reported as line comments, they are summarized here:

1. Remove this method "__construct" to simply inherit it.