Merge pull request #4229 from crazyserver/MOBILE-4694

MOBILE-4694 behat: Fix PHP 7.4 compatibility for LMS 4.1
moodlehq · Nov 6, 2024 · 07a1509 · 07a1509
2 parents d0a89e0 + 4496276
commit 07a1509
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/local_moodleappbehat/tests/behat/behat_app_helper.php b/local_moodleappbehat/tests/behat/behat_app_helper.php
@@ -740,7 +740,7 @@ protected function resize_app_window(int $width = 500, int $height = 720) {
      * @param string $text
      * @return string|string[] Transformed text.
      */
-    protected function transform_time_to_string(string $text): string|array {
+    protected function transform_time_to_string(string $text) {
         if (!preg_match('/##(.*)##/', $text, $matches)) {
             // No time found, return the original text.
             return $text;

diff --git a/local_moodleappbehat/version.php b/local_moodleappbehat/version.php
@@ -7,7 +7,7 @@
 defined('MOODLE_INTERNAL') || die;
 
 $plugin->version = {{ pluginVersion }};
-$plugin->requires = 2016052300;
+$plugin->requires = 2018051700;
 $plugin->maturity = MATURITY_STABLE;
 $plugin->release = '{{ appVersion }}';
 $plugin->component = 'local_moodleappbehat';
diff --git a/src/core/features/policy/tests/behat/consent.feature b/src/core/features/policy/tests/behat/consent.feature
@@ -21,7 +21,7 @@ Feature: Test accepting pending policies on signup
     But I should not be able to press "Continue" in the app
     And I should not be able to press "User account" in the app
 
-    When I press "Link to site policy agreement" in the app
+    When I press "Site policy agreement" "a" in the app
     And I press "OK" in the app
     Then the app should have opened a browser tab with url "moodle.org"