Project

Profile

Help

Bug #4170

closed

Wrong version information displayed for Saxon PHP module

Added by Frank Arensmeier over 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Category:
PHP API
Start date:
2019-03-16
Due date:
% Done:

100%

Estimated time:
Applies to branch:
Fix Committed on Branch:
Fixed in Maintenance Release:
Found in version:
1.1.2
Fixed in version:
1.2.0
SaxonC Languages:
SaxonC Platforms:
SaxonC Architecture:

Description

Freshly installed Saxon (1.1.2) PHP module (latest Saxon-HE/C for Linux) yield the following version information:

Saxon/C

Saxon/C => enabled
Saxon/C EXT version => 1.1.0
Saxon => 9.8.0.4
Excelsior JET => 11.3 MP1

Those values seem to be hardcoded in e.g. php_saxon.cpp. This is of course very confusing and it is really difficult to verify which version actually has been installed.

Actions #1

Updated by O'Neil Delpratt over 5 years ago

  • Category set to PHP API
  • Status changed from New to Resolved
  • Assignee set to O'Neil Delpratt
  • Priority changed from Low to Normal
  • % Done changed from 0 to 100
  • Found in version changed from Saxon-HEC1.1.2 to 1.1.2

Thank for spotting this bug.

Fixed and committed to subversion.

PHP_MINFO_FUNCTION(saxon)
{
    php_info_print_table_start();
    php_info_print_table_header(2, "Saxon/C", "enabled");
    php_info_print_table_row(2, "Saxon/C EXT version", "1.1.2");
    php_info_print_table_row(2, "Saxon", "9.8.0.15");
    php_info_print_table_row(2, "Excelsior JET", "15.3");
    php_info_print_table_end();
    DISPLAY_INI_ENTRIES();
}

See files to copy and update in your download: PHP7: https://dev.saxonica.com/repos/archive/opensource/latest9.8/hec/Saxon.C.API/php7_saxon.cpp PHP5: https://dev.saxonica.com/repos/archive/opensource/latest9.8/hec/Saxon.C.API/php_saxon.cpp

Are you building for PHP5 or PHP7?

Actions #2

Updated by Frank Arensmeier over 5 years ago

I am building for PHP7. Thank's for the quick response and have a great weekend!

Actions #3

Updated by O'Neil Delpratt over 5 years ago

Ok great. You can update the php7_saxon.cpp file. When building for php7 it will pickup the correct file.

Actions #4

Updated by O'Neil Delpratt about 5 years ago

  • Status changed from Resolved to Closed
  • Fixed in version set to 1.2.0

Big fix applied in the Saxon/C 1.2.0 release.

Please register to edit this issue

Also available in: Atom PDF