Bug #70070

Epic #69955: Optimize new Extbase/ Fluid based rewrite of EXT:form

Story #70068: EXT:form - Support bootstrap_package

EXT:form - Support attributes in form and container elements when using .layout

Added by Bjoern Jacob about 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Category:
Form Framework
Target version:
Start date:
2015-09-24
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

Setting the following layouts is currently not working. The class attribute is not set:


form (
    <form class="form-horizontal">
        <containerWrap />
    </form>
)
fieldset (
    <div class="fieldset"><fieldset class="fieldset"><legend /><containerWrap /></fieldset></div>
)

Associated revisions

Revision 413b5a06 (diff)
Added by Ralf Zimmermann about 4 years ago

[BUGFIX] EXT:form - class attribute in container elements is not shown

If a layout for FORM or FIELDSET is definded and it contains a
class attribute, the class attribute is not set in the template.

Resolves: #70070
Releases: master
Change-Id: Ib527cd269383932306805256415e273a76339cb6
Reviewed-on: http://review.typo3.org/43520
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Morton Jonuschat <>
Tested-by: Morton Jonuschat <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

#1 Updated by Bjoern Jacob about 4 years ago

  • Description updated (diff)

#2 Updated by Gerrit Code Review about 4 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#3 Updated by Benni Mack about 4 years ago

  • Sprint Focus set to Stabilization Sprint

#4 Updated by Gerrit Code Review about 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#5 Updated by Gerrit Code Review about 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#6 Updated by Gerrit Code Review about 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#7 Updated by Bjoern Jacob about 4 years ago

Example form for testing this issue.


prefix = tx_form
confirmation = 1

stdWrap.wrap = <div class="outerWrap">|</div>

postProcessor {
    1 = mail
    1 {
        recipientEmail = mail@email.com
        senderEmail = mail@email.com
        subject = Contactform website
        messages {
            success = TEXT
            success {
                value = Hartelijk dank voor uw reactie. Wij zullen uw vraag zo spoedig mogelijk beantwoorden.
            }
        }
    }
}

layout {
    containerWrap (
        <div class="containerWrap">
            <elements />
        </div>
    )
    elementWrap (
        <div class="elementWrap">
            <element />
        </div>
    )
    form (
        <form class="form-horizontal">
            <containerWrap />
        </form>
    )
    fieldset (
        <div class="fieldset"><fieldset class="fieldset"><legend /><containerWrap /></fieldset></div>
    )
}

10 = TEXTLINE
10 {
    name = name
    placeholder = Naam*
    label {
        value = Naam
    }
}
20 = TEXTLINE
20 {
    name = company
    placeholder = Bedrijf/organisatie
    label {
        value = Bedrijf/organisatie
    }
}
30 = TEXTLINE
30 {
    name = emailaddress
    placeholder = E-mailadres*
    label {
        value = Emailadres
    }
}
40 = TEXTLINE
40 {
    name = phonenumber
    placeholder = Telefoonnummer
    label {
        value = Telefoonnummer
    }
}
50 = TEXTAREA
50 {
    cols = 40
    rows = 10
    name = 5
    placeholder = Uw bericht...
    label {
        value = Uw bericht...
    }
}
60 = SELECT
60 {
    name = select
    label = Select
    10 = OPTION
    10 {
        name = option1
        value = option1
        data = Option 1
    }
    20 = OPTION
    20 {
        name = option2
        value = option2
        data = Option 2
    }
}
70 = CHECKBOX
70 {
    name = checkbox
    value = checkbox
    label = Checkbox
}
1000 = FIELDSET
1000 {
    legend = Submit
    10 = SUBMIT
    10 {
        name = 6
        value = Verzenden
    }
}
rules {
    1 = required
    1 {
        breakOnError = 0
        showMessage = 1
        message = *
        error = Dit veld is verplicht
        element = name
    }
    2 = required
    2 {
        breakOnError = 0
        showMessage = 1
        message = *
        error = Dit veld is verplicht
        element = emailaddress
    }
    3 = required
    3 {
        breakOnError = 0
        showMessage = 1
        message = *
        error = Dit veld is verplicht
        element = 5
    }
    4 = required
    4 {
        breakOnError = 0
        showMessage = 1
        message = *
        error = Dit veld is verplicht
        element = select
    }
    5 = required
    5 {
        breakOnError = 0
        showMessage = 1
        message = *
        error = Dit veld is verplicht
        element = checkbox
    }
}

#8 Updated by Gerrit Code Review about 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#9 Updated by Gerrit Code Review about 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#10 Updated by Gerrit Code Review about 4 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#11 Updated by Gerrit Code Review about 4 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#12 Updated by Gerrit Code Review about 4 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43520

#13 Updated by Ralf Zimmermann about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#14 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF