FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

PHP 8.6 support · Issue #1683 · microsoft/msphpsql · GitHub

Repository navigation

PHP 8.6 support #1683

Open
Open

Description

the last planned beta of php 8.6 was released yesterday.

would be nice if support for PHP 8.6 is released soon after php 8.6 release in november 2026

Activity

  1. David-Engel commented on Sep 14, 2026

    Collaborator

    That's the plan:

    https://github.com/microsoft/msphpsql/milestone/1

    Beta of the SQL drivers soon after PHP 8.6 RC1, then GA soon after PHP 8.6 GA.

  2. SharkMachine commented on Sep 24, 2026

    It doesn't build because of

    php_stream_wrapper_log_error(wrapper, options, "Invalid option: no options except REPORT_ERRORS may be specified with a sqlsrv stream");

    If you fix that, it builds and seems to work as well.

  3. SharkMachine commented on Sep 24, 2026

    I've only tested pdo_sqlsrv. There might be other issues with sqlsrv.

  4. Jan-E commented on Sep 24, 2026

    Fix for php_stream_wrapper_log_error is here:
    php/php-src#22958 (comment)

        if (options != 0) {
    #if PHP_VERSION_ID >= 80600
            php_stream_wrapper_log_error(wrapper, NULL, options, E_WARNING, 0, ZEND_ENUM_StreamErrorCode_Generic, "Invalid option: no options except REPORT_ERRORS may be specified with a sqlsrv stream");
    #else
            php_stream_wrapper_log_error(wrapper, options, "Invalid option: no options except REPORT_ERRORS may be specified with a sqlsrv stream");
    #endif
            return NULL;
        }
    
  5. jahnvi480 commented on Sep 25, 2026

    Contributor

    hebbet Jan Ehrhardt (@Jan-E) Petri Haikonen (@SharkMachine) even I have created one branch for testing the changes for PHP8.6 https://github.com/microsoft/msphpsql/tree/jahnvi/php86-driver-compatibility

    Below are the 2 files that need some change to bring in support for PHP8.6

    File Change
    source\\shared\\core_stream.cpp Added a PHP 8.6-specific call to the changed php_stream_wrapper_log_error() API, supplying the stream context and new error arguments. The older signature remains for earlier PHP versions. This shared code affects both drivers.
    source\\sqlsrv\\init.cpp Replaced the removed INI_BOOL / INI_INT macros with zend_ini_long(), preserving existing numeric parsing, boolean conversion and configuration behavior.

    We can share our learning and keep this thread posted. 🚀

  6. SharkMachine commented on Sep 25, 2026

    The fix is similar to the quick hack I made in shared\core_stream.cpp. It works for me 👍 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions


    Back | FazBrowse Home | New Git URL