fix: scoring state is now optional

Also remove subquestion classification for now, as it isn't used yet and has changed substantially.

classes/api/attempt_scored.php

@@ -30,27 +30,24 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 class attempt_scored extends attempt {
-    /** @var string */
-    public string $scoringstate;
+    /** @var string|null */
+    public ?string $scoringstate;
 
     /** @var string */
     public string $scoringcode;
 
     /** @var float|null */
     public ?float $score = null;
 
-    /** @var classified_response[]|null */
-    public ?array $classification = null;
-
     /**
      * Initializes a new instance.
      *
      * @param int $variant
      * @param attempt_ui $ui
-     * @param string $scoringstate
      * @param string $scoringcode
+     * @param string|null $scoringstate
      */
-    public function __construct(int $variant, attempt_ui $ui, string $scoringstate, string $scoringcode) {
+    public function __construct(int $variant, attempt_ui $ui, string $scoringcode, ?string $scoringstate = null) {
         parent::__construct($variant, $ui);
 
         $this->scoringstate = $scoringstate;