Project

General

Profile

Actions

Bug #11630

closed

Bitte check ob CALL() in 20.11.0 noch so funktioniert wie in 20.4.1

Added by Carsten Rose over 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
High
Assignee:
Carsten Rose
Target version:
Start date:
25.11.2020
Due date:
01.06.2022
% Done:

0%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

Description

Im IK-Tool #11625 hat das SQL 'CALL' noch funktioniert mit 20.4.1. Nach dem Update auf 20.11.0 funktioniert es nicht mehr.

In #11325 wurde CALL angeblich implementiert (fuer 20.11.0) - es muss aber vorher schon mal funktioniert haben! Und nun, nachdem es neu implementiert ist, funktioniert es nicht mehr. Das macht alles kein Sinn.

Code aus IK Tool:

    # Action: 'moveToPool'
    10 {
      sql = SELECT '' FROM (SELECT '') AS fake WHERE '{{action:SE}}' = 'moveToPool'

      10.sql = SELECT '' FROM (SELECT '') AS fake WHERE {{grIdPool:S0}}={{grIdPool2Bm:R}}
      10.10.sql = CALL copyApplicationPostingUpload(@srcId:={{apId:S0}})

      20.sql = UPDATE Application
                  SET grIdPool = {{grIdPool:S0}},
                      dateReceipt = IF( {{grIdPool:S0}}={{grIdPool2Bm:R}}, NOW(), dateReceipt),
                      finalCostCategory = IF({{grIdPool:S0}}={{grIdPool1Bm:R}},
                                             CASE
                                               WHEN totalCost=0 THEN ''
                                               WHEN totalCost<100000 THEN 'below 100k'
                                               WHEN totalCost<250000 THEN '100k to 250k'
                                               ELSE 'above 250k'
                                             END,
                                             finalCostCategory)
                  WHERE id = {{apId:S0}} AND '{{action:SE}}' = 'moveToPool'
    }

Befuerchtung:

  • In 20.4.1 war noch Code in QFQ das `CALL` funktioniert und dieser Code ist bis 20.11.0 verloren gegangen.
  • Warum funktioniert es nicht, obwohl CR das als 'fixed/implemented' markiert hat in 20.11.0.

Files


Related issues

Related to QFQ - Bug #11325: SQL: CALL() ClosedCarsten Rose16.10.2020

Actions
Actions #2

Updated by Carsten Rose over 3 years ago

Actions #4

Updated by Carsten Rose over 3 years ago

Es funktionert nicht (#11744).

Actions #5

Updated by Carsten Rose almost 3 years ago

  • Tracker changed from Support to Bug
Actions #6

Updated by Carsten Rose almost 3 years ago

  • Target version changed from next5 to next4
Actions #7

Updated by Carsten Rose almost 3 years ago

  • Target version changed from next4 to next3
Actions #8

Updated by Carsten Rose almost 3 years ago

  • Priority changed from High to Normal
Actions #9

Updated by Carsten Rose almost 3 years ago

  • Status changed from New to Priorize
Actions #10

Updated by Carsten Rose over 2 years ago

  • Assignee changed from Carsten Rose to Enis Nuredini
  • Priority changed from Normal to High

Check ob der Fehler noch auftritt. Wenn ja, bitte Ruecksprache mit CR.

Actions #11

Updated by Carsten Rose over 2 years ago

  • Target version changed from next3 to 21.12.0
Actions #12

Updated by Carsten Rose over 2 years ago

Hier ist ein Branch der nicht gemerged ist:

Actions #13

Updated by Carsten Rose over 2 years ago

  • Target version changed from 21.12.0 to 355
Actions #14

Updated by Enis Nuredini about 2 years ago

  • Status changed from Priorize to ToDo
Actions #15

Updated by Enis Nuredini about 2 years ago

  • Description updated (diff)
Actions #16

Updated by Enis Nuredini about 2 years ago

  • Status changed from ToDo to In Progress
Actions #17

Updated by Enis Nuredini about 2 years ago

  • Description updated (diff)
  • Status changed from In Progress to Feedback
  • Assignee changed from Enis Nuredini to Carsten Rose

Sollte mit dem Branch 11325_SQL_CALL gelöst sein. Merge Request wartend.

Actions #18

Updated by Carsten Rose almost 2 years ago

  • Due date set to 01.06.2022
  • Assignee changed from Carsten Rose to Enis Nuredini

Es ist kein Merge Request mehr offen - bitte:

  • Den aktuellen Develop Branch in diesen Branch mergen.
  • Check das die Unit Tests durchlaufen.
  • Neuen Merge Request machen.
  • Merge Request hier verlinken.

Danke.

Actions #19

Updated by Enis Nuredini over 1 year ago

  • Status changed from Feedback to Ready to sync (develop)
  • Assignee changed from Enis Nuredini to Carsten Rose
  • Prio Planung set to No

Das Ticket ging fast unter, hatte den Kommentar nicht gesehen.
Ich hab gesehen dass die wichtigen Änderungen spezifisch zu diesem Ticket schon im Dev Branch enthalten sind. Dieser Branch enthält nur noch eine Änderung des Kommentars in der DB Klasse. Für den sauberen Abschluss hab ich jetzt den Dev Branch zu diesem gemerged und die Unit Tests gecheckt.
Merge Request : https://git.math.uzh.ch/typo3/qfq/-/merge_requests/470

Actions #20

Updated by Carsten Rose over 1 year ago

  • Target version changed from 355 to 24.10.0
Actions #21

Updated by Carsten Rose about 1 year ago

  • Status changed from Ready to sync (develop) to Closed
  • Target version changed from 24.10.0 to 22.11.0
Actions

Also available in: Atom PDF