Hello,

if you do not select an image for a person, you get an fatal error, caused by the following line:
https://github.com/mindbird/contao-person/blob/master/src/Content/Person.php#L61

You simply have to check, if an image is set:

if ($objPerson->image) {
    \Controller::addImageToTemplate($this->Template, $arrData);
}

Cheers

Hi @Wusch

as you know we maintain a fork of your extension. Over time we added some features we need in every project. We would like to merge them into your extension to only have on repository to work with.

We would propose the following features:

• Image sizes Let editors decide which image size to use in person elements and modules and use this image size in the default templates
• Templates Add a dropown for a custom element or module template, like it is the default for core elements and modules since 3.3. There actually is PR #2 for that since 2016, which is now outdated obviously ;-) We would rebase our changes onto the new code structure and would resend a PR.
• Add a description field In every project we use this extension, we need a textarea for additional information, e.g. a short CV, personal statement. Adding this field to the core version, would help us enormously.

We think, these features come with no downside, because the default behaviour stays like it is atm.

Love to hear what you think :-)

Best
Benedict

Ein Content Element, welches eine Person ausgibt, anlegen.

If the user does not select a custom person template in person list module, there won't be any output. This is because the fallback or default value is not correctly used.

Will provide a fix with all the other commits in #6

Hello,

if you have a large number of persons, the select field in the Person content element can become quite unhandy. It helps a lot if you make it a Chosen field (https://github.com/mindbird/contao-person/blob/master/src/Resources/contao/dca/tl_content.php#L11-L13)

However, it helps even more if you group them by PersonArchive. We did all this here and we would like to refactor this and could then prepare a PR:
https://github.com/presprog/contao-person/blob/develop/contao/dca/tl_content.php#L9-L13
https://github.com/presprog/contao-person/blob/develop/contao/dca/tl_content.php#L34-L59

Tell us what you think :)

Cheers