$ 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

Comments

devdesagar created an issue. See original summary.

devdesagar’s picture

devdesagar commented
Issue summary: View changes
amsteri’s picture

amsteri commented

Set

short_open_tag=On

in php.ini

albapb’s picture

albapb commented
StatusFileSize
new geophp-php7-compatibility-3058149-4.patch223 bytes
albapb’s picture

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

dasginganinja
Primary language English
Location Bethlehem, PA
commented
Status: Needs review » Reviewed & tested by the community

The patch has been tested and works in our installation.

ruyakhokhar’s picture

ruyakhokhar commented
Assigned: Unassigned » ruyakhokhar
Issue tags: +PHP 7.3
StatusFileSize
new geophp-php7-compatibility-3058149-4_fixed.PNG42.67 KB

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

ruyakhokhar’s picture

ruyakhokhar commented
Assigned: ruyakhokhar » Unassigned

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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