Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-120548

QSqlQuery::positionalBindingEnabled improvements

    XMLWordPrintable

Details

    • b1e5d9275 (dev), 4bc230ad0 (6.7)

    Description

      API Review:

      • Should this be a positionalBindingEnabled property? Would make also the documentation more concise
      • Rename to isPositionalBindingEnabled(), setPositionalBindingEnabled()
      • Document default (true?)
      • Interlink with other methods related to positional bindings, class overview documentation.
      • Replace Postgres in documentaiton with PostgreSQL (official name)

      Attachments

        Issue Links

          blocks

          Task - A task that needs to be done. QTBUG-119952 Qt 6.7 API Change Review

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed
          relates to

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-96636 Some Postgresql jsonb operators mistaken for prepared statement placeholders.

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Closed
          For Gerrit Dashboard: QTBUG-120548
          # Subject Branch Project Status CR V
          528377,3 Doc: use official PostgreSQL name dev qt/qtbase Status: ABANDONED 0 0
          528424,6 SQL: rename enablePositionalBinding() to setPositionalBindingEnabled() dev qt/qtbase Status: MERGED +2 0
          529293,2 SQL: rename enablePositionalBinding() to setPositionalBindingEnabled() 6.7 qt/qtbase Status: MERGED +2 0

          Activity

            People

              chehrlic Christian Ehrlicher
              kkohne Kai Köhne
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes