- 🇳🇱Netherlands arantxio Dordrecht
The patch applies perfectly and the code changes for the functionality of the module seem to be working fine. I've not yet tested the test files that it changed, if I get to that I will update the issue to RTBC. If I cannot find the time someone should test those first before we can "close" this issue.
- 🇮🇳India abhinavk
I checked the patch #3 and it works fine. I just some errors while running drupal-check which are not ignorable, so adding fixes for them in new patch.
Also attaching drupal-check file. - 🇮🇳India Chandra Gowsalya Kannan
Tested the patch #6 but found warning when applying patch. So applied it manually and checked the upgrade status.
The module is compatible with D10 now.
git apply --check geophp_d10-3287733-6.patchwarning: geoPHP/tests/tests/20120702Test.php has type 100644, expected 100755
warning: geoPHP/tests/tests/adaptersTest.php has type 100644, expected 100755
warning: geoPHP/tests/tests/aliasesTest.php has type 100644, expected 100755
warning: geoPHP/tests/tests/geosTest.php has type 100644, expected 100755
warning: geoPHP/tests/tests/methodsTest.php has type 100644, expected 100755
warning: geoPHP/tests/tests/placeholdersTest.php has type 100644, expected 100755
warning: geophp.info.yml has type 100644, expected 100755
warning: geophp.module has type 100644, expected 100755 - Status changed to RTBC
over 1 year ago 10:19am 28 April 2023 - 🇮🇳India rahul1707
Tested patch #6 on Drupal 10.0.8 and 9.5.7 and PHP 8.1. Patch works perfectly fine for me and making this module D10 compatible. Checked with upgrade status as well which is giving no error found.
So, moving this to RTBC for D10 stable release. -
i-trokhanenko →
committed 6efaafea on 8.x-1.x
Issue #3287733 by AbhinavK: Automated Drupal 10 compatibility fixes
-
i-trokhanenko →
committed 6efaafea on 8.x-1.x
- Status changed to Fixed
over 1 year ago 7:14pm 10 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.