Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[RTM] Fixed Controller::getForm() #8527

Closed
wants to merge 4 commits into from
Closed

[RTM] Fixed Controller::getForm() #8527

wants to merge 4 commits into from

Conversation

Toflar
Copy link
Member

@Toflar Toflar commented Oct 14, 2016

which was relying on the Form class and always being rendered as a content element.

see contao/core-bundle#582

@aschempp
Copy link
Member

Why not just pass in the FQCN?

@Toflar
Copy link
Member Author

Toflar commented Oct 14, 2016

Because it does not make sense? You can only render it as CE or MOD. There is no other possibility.

@aschempp
Copy link
Member

somewhat true, yes. But it would be some sort of constant instead of a strange string.

@Toflar
Copy link
Member Author

Toflar commented Oct 14, 2016

¯\_(ツ)_/¯

@leofeyer
Copy link
Member

As discussed in Mumble on December 14th, @Toflar will change the argument to $blnModule and then the PR can be merged.

@Toflar
Copy link
Member Author

Toflar commented Dec 14, 2016

Updated.

@aschempp
Copy link
Member

aschempp commented Dec 14, 2016 via email

@leofeyer leofeyer added this to the 4.4.0 milestone Dec 14, 2016
@leofeyer leofeyer changed the title Fixed Controller::getForm() [RTM] Fixed Controller::getForm() Dec 14, 2016
leofeyer pushed a commit to contao/core-bundle that referenced this pull request Jan 19, 2017
@leofeyer leofeyer self-assigned this Jan 19, 2017
@leofeyer
Copy link
Member

Merged in contao/core-bundle@078cb34.

@leofeyer leofeyer closed this Jan 19, 2017
agoat pushed a commit to agoat/contao-core-bundle that referenced this pull request Apr 10, 2017
* Make Symfony 3.2 the minimum requirement (see contao#630).

* Set the encryption key from the kernel secret by default (see contao#660).

* Stop using the deprecated QuestionHelper::setInputStream() method.

* Update Dropzone to version 4.

* Prefer the caret operator over the tilde operator in the composer.json file.

* Add the contao.root_dir parameter (see contao#662).

* Update the change log.

* Stop using the contao-components/all meta package.

* Update the README.md file.

* Deprecate the contao:version command (see contao#668).

* Update the installation path.

* Auto-select the active page in the quick navigation/link module (see contao/core#8587).

* Look up the form class and allow to choose the type (see contao/core#8527).

* Add PHP Toolbox support (see contao#565).

* Remove the arrow brackets in the book navigation template (see contao/core#8607).

* Add a bottom padding to the buttons layer.

* Add the contao.web_dir parameter (see contao/installation-bundle#40).

* Fix the tests.

* Match security firewall based on request scope (see contao#677).

* Fix an issue found by Scrutinizer.

* Use the contao.web_dir parameter in the Combiner (see contao#679).

* Fix the tests.

* Add stripRootDir() method to System class (see contao#683).

* Add the contao.image.target_dir parameter (see contao#684).

* The ContaoCoreExtension::overwriteImageTargetDir() is not deprecated.

* Support custom backend routes (see contao#512).

* Use the scope matcher instead of checking the request attribute (see contao#688).

* Replace every occurrence of $contaoFramework with $framework.

* Fix an issue found by Scrutinizer.

* Fix deprecations in unit tests (see contao#687).

* Added a DBAL field type for UUIDs (see contao#415).

* Support importing form field options from a CSV field (see contao#444).

* Fix the coding style and the unit tests.

* Add the Doctrine field type in the config.yml file.

doctrine:
    dbal:
        types:
            binary_string:
                class: "Contao\\CoreBundle\\Doctrine\\DBAL\\Types\\BinaryStringType"
                commented: true

* Add a basic unit test for the BackendCsvImportController class.

* Update the change log.

* Fix rebuilding the search index (see contao#689).

* Also handle „no origin“ and „empty origin“ in the CORS provider.

* Remove an unused use statement.

* Remove the security.yml file and update the README.md file.

* Improve the e-mail extraction in the text element (thanks to Martin Auswöger).

* Rename the Test namespace to Tests.

* Update the composer.json file.

* Update the .php_cs file.

* Raise the minimum PHP version to 5.6 (see contao#701).

* Support using objects in callback arrays (see contao#699).

* Use try-finally blocks to close all output buffers when downloading a file (see contao#714).

* Fix the coding style.

* Only prefix an all numeric alias when standardizing (see contao#707).

* Adjust the test namespaces.

* Allow to manually pass a value to any widget (see contao#674).

* Add a change log entry and fix the tests.

* Disable the picker buttons if the main window does not show a picker.

* Use the file manager instead of the file picker.

* Use the site structure instead of the page picker.

* Always show the selected nodes.

* Add the menu builder.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants