This is related to https://forum.kde.org/viewtopic.php?f=221&t=165454.

In D28809#655086, @piggz wrote:

Amazing to have someone using my work, can I ask what app you are using it in?

Updated required version of ECM to use ECMSetupQtPluginMacroNames. It is now
based on master (3.3) instead of the 3.2 branch

Fixed style issue

Change error message to be easier to translate and to understand, taking care that quoting characters are not quoted

Fix summary

Remove the unrelated change for indentifier escaping

Add comment to fallback rules that now report an actual error

Beware that the SQL query in the summary will not work without the change in D29277 and it will fail with

In D29277#659789, @staniek wrote:

Related fix in KEXI 3.2 would be welcome.

In D29277#659787, @staniek wrote:

Thanks. Is there any relation to https://bugs.kde.org/show_bug.cgi?id=420599?

Fix indent

Here are a couple of GIF showing the issue before and after the changes in this commit.

My plan is to include the example as a tutorial in the documentation, but first i wanted for you to review the code alone and see all i did wrong.

In D28809#652442, @staniek wrote:

BTW, while creating reviews please add at least me and KReport maintainer Adam as reviewers. Thanks!

Improve wording, remove use of QLatinString/QStringLiteral, and make code follow conventions better

Fix wording, and remove improper use of auto and pointer validity according to coding style

Updated text according to Jarosław’s notes

In D28736#645582, @staniek wrote:

If we want it in the stable 3.2 release, it needs to land in the 3.2 branch.