Resolved implicitly marking parameter as nullable deprecation

Author: lucaboesch

classes/question/question.php

@@ -653,9 +653,9 @@ public function update($questionrecord = null, $updatechoices = true) {
      * Add the question to the database from supplied arguments.
      * @param \stdClass $questionrecord The required data for adding the question.
      * @param array $choicerecords An array of choice records with 'content' and 'value' properties.
-     * @param boolean $calcposition Whether or not to calculate the next available position in the survey.
+     * @param bool|null $calcposition Whether or not to calculate the next available position in the survey.
      */
-    public function add($questionrecord, array $choicerecords = null, $calcposition = true) {
+    public function add($questionrecord, ?array $choicerecords = null, ?bool $calcposition = true) {
         global $DB;
 
         // Create new question.

classes/question/rate.php

@@ -1026,9 +1026,9 @@ public function insert_nameddegrees(array $nameddegrees) {
      * Helper function used to move existing named degree choices for the specified question from the "quest_choice" table to the
      * "question" table.
      * @param int $qid
-     * @param null|\stdClass $questionrec
+     * @param \stdClass|null $questionrec
      */
-    public static function move_nameddegree_choices(int $qid = 0, \stdClass $questionrec = null) {
+    public static function move_nameddegree_choices(int $qid = 0, ?\stdClass $questionrec = null) {
         global $DB;
 
         if ($qid !== 0) {
@@ -1079,7 +1079,7 @@ public static function move_nameddegree_choices(int $qid = 0, \stdClass $questio
      * questionnaire before '2018110103'.
      * @param int|null $surveyid
      */
-    public static function move_all_nameddegree_choices(int $surveyid = null) {
+    public static function move_all_nameddegree_choices(?int $surveyid = null) {
         global $DB;
 
         // This operation might take a while. Cancel PHP timeouts for this.

classes/responsetype/responsetype.php

@@ -45,9 +45,9 @@ abstract class responsetype {
      * responsetype constructor.
      * @param \mod_questionnaire\question\question $question
      * @param int|null $responseid
-     * @param array $choices
+     * @param array|null $choices
      */
-    public function __construct(\mod_questionnaire\question\question $question, int $responseid = null, array $choices = []) {
+    public function __construct(\mod_questionnaire\question\question $question, ?int $responseid = null, ?array $choices = []) {
         $this->question = $question;
         $this->responseid = $responseid;
         $this->choices = $choices;

tests/behat/behat_mod_questionnaire.php

@@ -418,7 +418,7 @@ private function add_response_data($qid, $sid) {
      *                         to replace with mapped values.
      * @return null
      */
-    private function add_data(array $data, $datatable, $mapvar = '', array $replvars = null) {
+    private function add_data(array $data, $datatable, $mapvar = '', ?array $replvars = null) {
         global $DB;
 
         if ($replvars === null) {

tests/generator/lib.php

@@ -76,10 +76,10 @@ public function questionnaires() {
     /**
      * Create a questionnaire activity.
      * @param array $record Will be changed in this function.
-     * @param array $options
+     * @param array|null $options
      * @return questionnaire
      */
-    public function create_instance($record = null, array $options = null) {
+    public function create_instance($record = null, ?array $options = null) {
         $record = (object)(array)$record;
 
         $defaultquestionnairesettings = array(