Formatter doesn't handle CASE expressions and statements

Open

#274 opened on Dec 15, 2019 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 4/5
Estimated time: 3-5 days
Newbie friendliness: 45/100

Issue type: Bug
Clarity: Mostly clear
Activity status: Stale
Tech stack: mysql, php
Domain: databases

Research direction

Use the MySQL CASE expression and CASE statement references, plus the two formatting examples, as the acceptance cases. Compare the current formatter behavior with related issue #272, then locate the formatter entry point and add coverage for both forms; done means each example retains readable CASE and statement clause indentation.

Written by the indexing model from the issue text.

Description

bug

Formatter doesn't handle CASE expressions and CASE statements.

Related to #272.

Examples:

CASE expression:

-- original
SELECT CASE 1
    WHEN 1 THEN
        'one'
    WHEN 2 THEN
        'two'
    ELSE
        'more'
END;

-- formatted
SELECT CASE
    1 WHEN 1 THEN 'one' WHEN 2 THEN 'two' ELSE 'more'
END;

CASE statement:

-- original
CREATE PROCEDURE p()
BEGIN
    DECLARE
        v INT DEFAULT 1;
    CASE v
        WHEN 2 THEN
            SELECT v;
        WHEN 3 THEN
            SELECT 0;
        ELSE
            BEGIN
            END;
    END CASE;
END;

-- formatted
CREATE PROCEDURE p()
BEGIN
    DECLARE
        v INT DEFAULT 1; CASE v WHEN 2 THEN
    SELECT
        v; WHEN 3 THEN
    SELECT
        0; ELSE
    BEGIN
        END
            ;
    END CASE;
END;

Dominant language: PHP
Stars: 485
Forks: 119
PR merge metrics: No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

Admissions heading updates Open

a11y admissions.uiowa.edu needs grooming SiteImprove best practice

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

uiowa/uiowa#10137 · Opened Sep 22, 2026
Save States Menu Open

Difficulty 2/5 1-3 hours Newbie friendliness 68/100

RobLoach/nextcloud-nostalgist#19 · Opened Sep 22, 2026
Error on exporting the error log Open

Difficulty 2/5 1-3 hours Newbie friendliness 84/100

pluginsGLPI/datainjection#673 · Opened Sep 22, 2026
RecordImport::importData()` ignores the "Update Existing Records" checkbox when importing entries into a Tailor blueprint Open

Difficulty 2/5 1-3 hours Newbie friendliness 82/100

octobercms/october#6130 · Opened Sep 22, 2026
`file` and `filepond` fields register jQuery outside the `bottom` asset group Open

Difficulty 2/5 1-3 hours Newbie friendliness 82/100

getgrav/grav-plugin-form#656 · Opened Sep 22, 2026

More PHP issues