@yabwon wrote:
I don't remember which @Kurt_Bremser 's maxim it was, but I'm guessing like first: "Read the log"
Actually, it's the second. The first is "Read the Documentation".
But we also have a violation of another maxim here, #34: "Work in Steps". By not building the SQL peacemeal, element by element, the OP created code with so many ERROR messages at once that the most important one was kind of "drowned out" and needed expert's eyes to be discovered.
We also see a violation of Maxim 12, "Make it Look Nice".
There is no consistent use of uppercase/lowercase, in fact most of the code is "shouting at the compiler", which makes it hard to read in the first place. There is no indentation, no use of blanks to separate words and operators, and complex elements like the case blocks are kept in one line, making it hard to see the branches.
Bottom line: this is an example for "how not to do it". That the code fails in a rather spectacular way should not come as a surprise.
... View more