mahara-contributors team mailing list archive

[Mahara Bot <1976422@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/c/mahara/+/12889
Committed: https://git.mahara.org/mahara/mahara/commit/0b582e96060325e511b0f2acbb9b0b045f19f038
Submitter: "Robert Lyon <robertl@xxxxxxxxxxxxxxx>"
Branch:    main

commit 0b582e96060325e511b0f2acbb9b0b045f19f038
Author: Gold <gold@xxxxxxxxxxxxxxx>
Date:   Fri Jun 17 11:56:26 2022 +1200

Bug 1976422: PHP 8 Compatibility: json

Added default values for many vars.
Refactored relatedtagssearch.php to allow $view to be unambiguously
available later in the script.
Corrected the @return in lib/view.php for static analysis.
Removed htdocs/json/search.php as the file that used it was removed
in commit e7619c78c6e828aacd08cbde8b38e6a6da3e5268

Signed-off-by: Gold <gold@xxxxxxxxxxxxxxx>
Change-Id: I996372b033e05bfdbe01bffc135383ee0590b13d

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1976422

Title:
  PHP 8 Compatibility: json

Status in Mahara:
  Fix Committed

Bug description:
  This is to track patchsets that will bring the codebase up to be
  compliant with PHP 8.1.

  The initial run through will be on a top level directory basis. i.e.
  the directories below htdocs. It will focus on the results from the
  PHPStan and Phan static analysis tools.

  This bug tracks files in the json directory.

  Testing will require identifying, from the patchset, where the code is
  used and then verifying that that area of the site still works in both
  PHP 7.4 and PHP 8.1.

  https://phpstan.org/
  https://github.com/phan/phan

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1976422/+subscriptions