Ruudt / contao-page_images

Contao extention to set one or more images per page.
0 stars 5 forks source link

does "defaultImages" has to be be mandatory? #9

Closed gingebaker closed 11 years ago

gingebaker commented 11 years ago

Hi I was wondering if you see a change to put the DefaultImage Field $GLOBALS['TL_DCA']['tl_pageimages']['fields']['multiSRC'] to mandatory = false ? https://github.com/Ruudt/contao-page_images/blob/master/dca/tl_pageimages.php#L148

I sometimes have the need to have no Image rendered at all... Wich is actually only possible by changing the DCA. (correct me if i´m wrong) With mandatory set to false it would be just a bit more flexible... but that´s only my opinion However, thanks for the great module...

Ruudt commented 11 years ago

No, I guess it doesn't. Module will return en empty string if no image is found.