David Bruchmann comments

Results 155 comments of


                                            David Bruchmann

List cms-package-dir as no longer being used property

`cms-package-dir` is indeed outdated since long time. I found it only in TYPO3 v7.4 and v8, but in v9 it was already removed. Based on `TYPO3\CMS\Composer\Installer\CoreInstaller` it seems to me...

List cms-package-dir as no longer being used property

To make it more detailed: the following code is included in the `CoreInstaller`: ``` self::$deprecationShown = true; $this->io->writeError('Config option "cms-package-dir" has not been set or set to a value different...

Revert the change "fix package name for cms_rte_ckeditor in ext_emconf.php"

In the failing tests / github "jobs" for v13 is still `typo3/cms-recordlist` mentioned, that was already removed in v12.

selectImage multiple selection

For the output a single attribute `width` or `height` is criticized by lighthouse. So, no matter if the image is processed or not (with an option `noScale`), the missing value...

width and height without reduction

In TypoScript the option `noScale` is used: https://docs.typo3.org/m/typo3/reference-typoscript/12.4/en-us/Functions/Imgresource.html#noscale I propose to use the same option name.

Using images not from the file system

shouldn't the storage be responsible to deliver the domain?

Using images not from the file system

ok, then I misunderstood the problem.

explode() - string required, int passed

I just patched the GeneralUtility for it: ``` - $result = explode($delimiter, $string); + $result = explode($delimiter, (string) $string); ```

Relation RenderType missing

I propose a label `GUI` for those issues

Saving and reopening (simple) new extension shows undefined properties

@bytesinmotion I created a folder with the extension key and moved your json file inside. After this I could open and also change your mini-extension. I added the field `description`...