Uploaded image for project: 'phpBB3'
  1. phpBB3
  2. PHPBB3-10016

set_config_count() fails on PostreSQL 7

    Details

    • Type: Bug
    • Status: Unverified Fix
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0.9-RC1
    • Component/s: None
    • Labels:
      None
    • Environment:
      PostgreSQL 7.4.23

      Description

      Reported by Human-Bean on IRC.

      General Error
       
      SQL ERROR [ postgres ]
       
      ERROR: cannot cast type character varying TO numeric []
       
       
      SQL
       
      UPDATE phpbb_movieconfig SET config_value = CAST(CAST(config_value AS DECIMAL(255, 0)) + 3 AS VARCHAR(255)) WHERE config_name = 'num_users'
      

        Issue Links

          Activity

          Hide
          Oleg Oleg [X] (Inactive) added a comment -

          Definitely a step in the right direction.

          I am giving up on trying to get firebird (at least 2.1) working. If someone provides instructions for setting it up on some reasonably common linux distro for use with phpbb I will pay hourly charges to run linux on aws to test this.

          Show
          Oleg Oleg [X] (Inactive) added a comment - Definitely a step in the right direction. I am giving up on trying to get firebird (at least 2.1) working. If someone provides instructions for setting it up on some reasonably common linux distro for use with phpbb I will pay hourly charges to run linux on aws to test this.
          Hide
          Oleg Oleg [X] (Inactive) added a comment -

          This bug renders 3.0.8 unusable on postgres 7.3 and presumably 7.4 as posting produces the listed database error.

          Show
          Oleg Oleg [X] (Inactive) added a comment - This bug renders 3.0.8 unusable on postgres 7.3 and presumably 7.4 as posting produces the listed database error.
          Hide
          Oleg Oleg [X] (Inactive) added a comment - - edited

          Posting is possible with the patch applied on 7.3.21.

          Postgres 9.0.3 looks to still work too.

          Show
          Oleg Oleg [X] (Inactive) added a comment - - edited Posting is possible with the patch applied on 7.3.21. Postgres 9.0.3 looks to still work too.
          Hide
          Oleg Oleg [X] (Inactive) added a comment - - edited

          It looks like the fix for develop above is actually correct since it is only for postgres.

          Regarding backporting the dbal code to 3.0, is that something we really want to do?

          The merging strategy here would be to merge ticket/10016 into develop-olympus, then merge develop-olympus into develop reverting ticket/10016, and finally merge ticket/10016-develop into develop.

          Show
          Oleg Oleg [X] (Inactive) added a comment - - edited It looks like the fix for develop above is actually correct since it is only for postgres. Regarding backporting the dbal code to 3.0, is that something we really want to do? The merging strategy here would be to merge ticket/10016 into develop-olympus, then merge develop-olympus into develop reverting ticket/10016, and finally merge ticket/10016-develop into develop.
          Hide
          Oleg Oleg [X] (Inactive) added a comment -

          The two pull requests:

          https://github.com/phpbb/phpbb3/pull/122 (develop-olympus)
          https://github.com/phpbb/phpbb3/pull/123 (develop)

          When merging develop-olympus into develop the conflict may be resolved as follows:

          git reset HEAD phpBB/includes/functions.php
          git co -- phpBB/includes/functions.php     
          git ci                                     
          

          Show
          Oleg Oleg [X] (Inactive) added a comment - The two pull requests: https://github.com/phpbb/phpbb3/pull/122 (develop-olympus) https://github.com/phpbb/phpbb3/pull/123 (develop) When merging develop-olympus into develop the conflict may be resolved as follows: git reset HEAD phpBB/includes/functions.php git co -- phpBB/includes/functions.php git ci

            People

            • Assignee:
              Oleg Oleg [X] (Inactive)
              Reporter:
              bantu Andreas Fischer
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development