Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.x
    • Fix Version/s: 3.0.10-RC1
    • Component/s: Installation system
    • Labels:
      None
    • Environment:
      PHP Environment: 5.2.10
      Database: MySQL 5.0.67

      Description

      When updating phpbb 3.0.5 to 3.0.6-RC1 it get:

      Result :: Some queries failed, the statements and errors are listing below.
       
          * Error :: You have an error in your SQL syntax; check the manual that corresponds to your
             MySQL server version for the right syntax to use near 'log(log_time)' at line 1
            SQL :: CREATE INDEX log_time ON log(log_time)
      

      I assume this is because LOG is a mysql function and that I don't have any table prefixs.

      I tried this which worked:

          CREATE INDEX log_time ON `log`(log_time);
      

        Issue Links

          Activity

          Hide
          Eelke Eelke Blok [X] (Inactive) added a comment -

          I don't believe phpBB3 is meant to be run without a table prefix. Possibly, all that is needed is a sanity check on the table prefix, at install time. I would recommend to change your database so that a prefix is used.

          Show
          Eelke Eelke Blok [X] (Inactive) added a comment - I don't believe phpBB3 is meant to be run without a table prefix. Possibly, all that is needed is a sanity check on the table prefix, at install time. I would recommend to change your database so that a prefix is used.
          Hide
          Acyd Burn Meik Sievertsen [X] (Inactive) added a comment -

          But it should work without a table prefix. I will think about it's severity a bit.

          Show
          Acyd Burn Meik Sievertsen [X] (Inactive) added a comment - But it should work without a table prefix. I will think about it's severity a bit.
          Hide
          Acyd Burn Meik Sievertsen [X] (Inactive) added a comment -

          Ok, we should add a check to the installation to not allow empty prefixes because this will not only give problems with the index creation, but also with table creation, column changes, column additions, etc.

          Show
          Acyd Burn Meik Sievertsen [X] (Inactive) added a comment - Ok, we should add a check to the installation to not allow empty prefixes because this will not only give problems with the index creation, but also with table creation, column changes, column additions, etc.
          Hide
          bantu Andreas Fischer added a comment -

          Just dropped the index creation statement. That means it should no longer happen on new installs. I'll leave this open because of the empty table prefix thing.

          Show
          bantu Andreas Fischer added a comment - Just dropped the index creation statement. That means it should no longer happen on new installs. I'll leave this open because of the empty table prefix thing.
          Hide
          bantu Andreas Fischer added a comment -

          One possible solution would be enforcing that table prefix is never empty.

          Show
          bantu Andreas Fischer added a comment - One possible solution would be enforcing that table prefix is never empty.
          Hide
          bantu Andreas Fischer added a comment -

          The problem is log(log_time) is interpreted as the logarithm function applied to log_time.

          Show
          bantu Andreas Fischer added a comment - The problem is log(log_time) is interpreted as the logarithm function applied to log_time.

            People

            • Assignee:
              bantu Andreas Fischer
              Reporter:
              aweiredguy aweiredguy
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development