Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

includeBlankOption does not work at select #63

Open
zonky2 opened this issue Nov 26, 2019 · 13 comments
Open

includeBlankOption does not work at select #63

zonky2 opened this issue Nov 26, 2019 · 13 comments
Assignees
Labels
wontfix This will not be worked on
Milestone

Comments

@zonky2
Copy link
Contributor

zonky2 commented Nov 26, 2019

includeBlankOption does not work at select

0 is selected

            'mcw_select' => array
                (
                'label'         => &$GLOBALS['TL_LANG'][$strTblName]['mcw_select'],
                'exclude'       => true,
                'inputType'     => 'select',
                'options'       => array(0,1,2,3,4,5,6,7,8,9,10),
                'eval'          => array
                    (
                        'style'              => 'width:150px',
                        'includeBlankOption' => true,
                        'chosen'             => false,
                    ),
                ),
@zonky2 zonky2 added the bug Something isn't working label Nov 26, 2019
@zonky2 zonky2 added this to the 3.4.3 milestone Nov 26, 2019
@zonky2
Copy link
Contributor Author

zonky2 commented Nov 27, 2019

includeBlankOptionworks fine if the first value not 0 - looks like Contao made a bug...

@discordier
Copy link
Member

I suspect this is the usual '' == 0 == null problem we have in Contao.

@zonky2
Copy link
Contributor Author

zonky2 commented Dec 12, 2019

@zonky2 zonky2 added wontfix This will not be worked on and removed bug Something isn't working labels Dec 12, 2019
@discordier
Copy link
Member

So we can close here and refer to the upstream ticket?

@zonky2
Copy link
Contributor Author

zonky2 commented Dec 12, 2019

I`ll close this, if in contao fixed...

@stefanheimes stefanheimes modified the milestones: 3.4.4, 3.4.5 Mar 8, 2020
@zonky2 zonky2 modified the milestones: 3.4.5, 3.4.x Sep 9, 2020
zonky2 referenced this issue in zonky2/contao Sep 24, 2020
Fix includeBlankOption with value 0 - issue contao#1022

implement contao#1022 (comment) with cast option values to string if values int - see https://github.com/menatwork/contao-multicolumnwizard-bundle/issues/63
@Total-Reality
Copy link

Mal ehrlich... welchen Sinn ergibt es bei den Options eine 0 zu übergeben und gleichzeitig includeBlankOption?

@zonky2
Copy link
Contributor Author

zonky2 commented Dec 5, 2020

weil es einen Unterschied beim Programmieren zwischen null und 0 gibt... und ja, den usecase gibt es!

@Total-Reality
Copy link

Hab ich noch nie gebraucht und ich hab schon über 60 Erweiterungen programmiert. Also nenn mir doch bitte mal einen Anwendungsfall wo man das braucht.

@zonky2
Copy link
Contributor Author

zonky2 commented Dec 8, 2020

MM Levenshtein-Filter

@Total-Reality
Copy link

What?

@zonky2
Copy link
Contributor Author

zonky2 commented Dec 8, 2020

@Total-Reality
Copy link

Levenshtein kenn ich, du hattest das aber irgendwie anders geschrieben. Außerdem sagte mir MM nix, hab MetaModels auch noch nie verwendet. Wir programmieren wie gesagt alle Erweiterungen selbst.

@zonky2
Copy link
Contributor Author

zonky2 commented Aug 29, 2021

Wir programmieren wie gesagt alle Erweiterungen selbst.

gern einen eigenen MCW wo man die 0im Select nicht benötigt ;-)

zonky2 referenced this issue in zonky2/contao Sep 1, 2021
Fix includeBlankOption with value 0 - issue contao#1022

implement contao#1022 (comment) with cast option values to string if values int - see https://github.com/menatwork/contao-multicolumnwizard-bundle/issues/63
zonky2 referenced this issue in zonky2/contao Sep 1, 2021
implement contao#1022 (comment) with cast option values to string if values int - see https://github.com/menatwork/contao-multicolumnwizard-bundle/issues/63

| Q                | A
| -----------------| ---
| Fixed issues     | Fixes contao#1022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

4 participants