ruudt / contao-page_images Goto Github PK

When the selected image for any row is removed it can't be shown. If all images where removed the fallback is not shown. Similar to #5

When selecting more than one page in the page selector and saving the entry, only the first selected page is remembered. All other pages are dropped.

Hi Ruudt, thanks for this great extension
...could be that the extension don't support svg-images (Ftontend)?

Blank pop up screen when trying to select an image or to reference to a specific page.
Contao Version 3.4

Do you have any plans to publish any version which is compatible to Contao 4 LTS?

Icon of module not visible in menu

Hi there,
I created a category and added a page image for the site root.
Then I created a module with the above category checked and put it into my theme layout. But there is no html rendering and no error.

In the module i noticed i can pick only one template.

Am I doing something wrong?

Thank you!

After running the 3.2 update scripts from a 3.1.x site I ran into the following error:

Warning: Invalid argument supplied for foreach() in system/modules/pageimages/classes/PageImages.php on line 137

I've narrowed the problem down to get images line 187:

$objFiles = \FilesModel::findMultipleByUuids($multiSRC);

If i replace this with:

$objFiles = \FilesModel::findMultipleByIds($multiSRC);

Then it works otherwise it returns the contao 2 format error, which ends up in the warnings because it's not expecting a string. I think this is because images are still using ids and not uuids because they do not get updated. Probably for reasons similar to the ones discussed here:
contao/core#6481
contao/core#6459
Since page image uses it's own table instead of a core table it probably doesn't get uuids while the update script runs.

This could be solved by either running the update on the page image files via the runonce or by using findMultipleByIds which will validate the uuid and use findMultipleByUuids if valid automatically.

If a page gets more then one images assigned through more then one record then some images may not show.

Warning: Division by zero in system/modules/pageimages/dca/tl_pageimages_items.php on line 304

#0 system/modules/pageimages/dca/tl_pageimages_items.php(304): __error(2, 'Division by zer...', '....', 304, Array)
#1 system/modules/pageimages/dca/tl_pageimages_items.php(242): tl_pageimages_items->getImageHTML(Object(Contao\File))
#2 system/modules/pageimages/dca/tl_pageimages_items.php(183): tl_pageimages_items->getRandomImages('a:1:{i:0;i:22;}')
#3 system/modules/core/drivers/DC_Table.php(3840): tl_pageimages_items->showLabel(Array)
#4 system/modules/core/drivers/DC_Table.php(351): Contao\DC_Table->parentView()
#5 system/modules/core/classes/Backend.php(424): Contao\DC_Table->showAll()
#6 contao/main.php(129): Contao\Backend->getBackendModule('pageimages')
#7 contao/main.php(271): Main->run()
#8 {main}

Probable cause is that no valid images are selected (I found a single corrupt jpeg in the selected folder)

If only an image folder is selected, images in this folder won't show up.
Tested with version 3.1.0.

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...

It's a great feature to inherit page images to subpages. Is there a chance to have an additional 'exlude' page tree?

Example:

• I want every page to show a page image. I selected the root page and activated inheritance. Works fine. This root image works as a fallback for all pages, that do not have a special page image defined.
• I now want the start page (and maybe one or two other pages) to not include a page image.

Contao shows just white screen.
Error log shows: PHP Fatal error: Cannot redeclare class tl_module_pageimages in /.../contao/system/modules/pageimages/dca/tl_module.php on line 73

I found in system/modules there are now 2 folders, one is called pageimages, the other is called page_images. The content is identical. ???
This double content is also in the zip on https://contao.org/de/extension-list/view/page_images.html

page_images version was 30000019 build 3

For now I fixed it by just putting a .skip file into page_images folder.

Group permission settings are not showing

Add option to show all images in module

Option requested and reference code by Michael Nußbaumer. Thanks!

After installation on Contao 3.2.2. the following errormessage pops up.

Fatal error: Uncaught exception Exception with message Incomplete relation defined for tl_pageimages_items.pages thrown in      system/modules/core/library/Contao/DcaExtractor.php on line 317