$ phpcs --standard=PHPCompatibility --extensions=php,module,inc,install,test,profile,theme,css --runtime-set testVersion 7.2.

FILE: \sites\default\modules\contrib\geophp\geoPHP\tests\postgis.php
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
1 | WARNING | No PHP code was found in this file and short open tags are not allowed by this install of PHP. This file may be using short open tags but PHP does not allow them.
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

CommentFileSizeAuthor
#7 geophp-php7-compatibility-3058149-4_fixed.PNG42.67 KBruyakhokhar
#4 geophp-php7-compatibility-3058149-4.patch223 bytesalbapb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

devdesagar created an issue. See original summary.

devdesagar’s picture

devdesagar CreditAttribution: devdesagar commented
Issue summary: View changes
Amsteri’s picture

Amsteri CreditAttribution: Amsteri as a volunteer and at Drudesk commented

Set

short_open_tag=On

in php.ini

albapb’s picture

albapb CreditAttribution: albapb commented
FileSize
geophp-php7-compatibility-3058149-4.patch223 bytes
albapb’s picture

albapb CreditAttribution: albapb commented
Status: Active » Needs review
dasginganinja’s picture

dasginganinja
Primary language English
Location Bethlehem, PA
CreditAttribution: dasginganinja as a volunteer and at Lehigh University commented
Status: Needs review » Reviewed & tested by the community

The patch has been tested and works in our installation.

ruyakhokhar’s picture

ruyakhokhar CreditAttribution: ruyakhokhar at TATA Consultancy Services for Pfizer, Inc. commented
Assigned: Unassigned » ruyakhokhar
Issue tags: +PHP 7.3
FileSize
geophp-php7-compatibility-3058149-4_fixed.PNG42.67 KB

Hi all, Patch#4 works fine for me. Screenshot attached.

ruyakhokhar’s picture

ruyakhokhar CreditAttribution: ruyakhokhar at TATA Consultancy Services for Pfizer, Inc. commented
Assigned: ruyakhokhar » Unassigned

Neslee Canil Pinto’s picture

Neslee Canil Pinto
he/him
Primary language English
CreditAttribution: Neslee Canil Pinto at Specbee commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.