Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-34081

Install fails on mssql if using binary collation

          Details

          • Testing Instructions:
            Hide

            You need a mssql server with binary collation, mine was Latin1_General_BIN. You must be using the freedts driver (mssql, not sqlsrv)

            1. Attempt to install
            2. run phpunit tests
            Show
            You need a mssql server with binary collation, mine was Latin1_General_BIN. You must be using the freedts driver (mssql, not sqlsrv) Attempt to install run phpunit tests
          • Affected Branches:
            MOODLE_23_STABLE
          • Fixed Branches:
            MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
          • Pull from Repository:
          • Pull Master Branch:
            mssql-case-sensitive
          • Pull Master Diff URL:

            Description

            In MSSQL using the freedts driver with a binary collation, Moodle fails to install due to problems with information_schema.

            This can be resolved by using the proper uppercase naming.

              Gliffy Diagrams

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  skodak Petr Skoda added a comment -

                  +1 these should be already fixed and tested in the sqlsrv driver

                  Show
                  skodak Petr Skoda added a comment - +1 these should be already fixed and tested in the sqlsrv driver
                  Hide
                  Permalink
                  poltawski Dan Poltawski added a comment -

                  Yep, the sqlsrv driver is using uppercase.

                  Show
                  poltawski Dan Poltawski added a comment - Yep, the sqlsrv driver is using uppercase.
                  Hide
                  Permalink
                  stronk7 Eloy Lafuente (stronk7) added a comment -

                  Integrated (21, 22, 23 and master), thanks!

                  Show
                  stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (21, 22, 23 and master), thanks!
                  Hide
                  Permalink
                  stronk7 Eloy Lafuente (stronk7) added a comment -

                  Installs OK now and runs phpunit tests OK too. Only the 2 well-known "mssql_data_seek" exceptions are shown. So passing.

                  Show
                  stronk7 Eloy Lafuente (stronk7) added a comment - Installs OK now and runs phpunit tests OK too. Only the 2 well-known "mssql_data_seek" exceptions are shown. So passing.
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Congratulations your code is upstream - gold star for you!

                  This issue + 79 others made it in in time for the minor releases.
                  Thank you everyone involved for your exuberant efforts.

                  Show
                  samhemelryk Sam Hemelryk added a comment - Congratulations your code is upstream - gold star for you! This issue + 79 others made it in in time for the minor releases. Thank you everyone involved for your exuberant efforts.

                    People

                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      1 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved:
                        Fix Release Date:
                        9/Jul/12