Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

DataObjectManager Module /

Discuss the DataObjectManager module, and the related ImageGallery module.

Moderators: martimiz, UncleCheese, Sean, biapar, Willr, Ingo, swaiba, simon_w

Bug Reports


Reply


297 Posts   52858 Views

Avatar
Martijn

Community Member, 271 Posts

30 August 2009 at 3:11am

Edited: 30/08/2009 3:12am

When changing the Gallery name the Folder Title always stays NewImageGalleryPage, so when people browse images from the TinyMCE image button, they see x times NewImageGalleryPage.

Fix :

ImageGalleryPage.php

add:

function onBeforeWrite() {
      parent::onBeforeWrite();
      if( $this->ID )
         $this->RootFolder()->Title = $this->Title;
// or $this->MenuTitle?      
   }

Avatar
UncleCheese

Forum Moderator, 4096 Posts

1 September 2009 at 6:24am

Is there a specific block of code you're referring to?

Avatar
UncleCheese

Forum Moderator, 4096 Posts

1 September 2009 at 7:38am

All that code is lifted from HasManyComplexTableField, actually. Probably ought to submit that bug to them, too. Thanks.

Avatar
Willr

Forum Moderator, 5513 Posts

1 September 2009 at 5:16pm

Yes notices to do with 5.3 have been fixed in 2.3.3 and trunk. The actual root cause of these notices will be fixed in 2.4 etc but they might be too intense to get into 2.3 so hence we just hide the notices.

Avatar
UncleCheese

Forum Moderator, 4096 Posts

6 September 2009 at 9:46am

I've checked in a version of these files using is_object instead of get_class.

Avatar
UncleCheese

Forum Moderator, 4096 Posts

6 September 2009 at 9:47am

The ImageGallery RootFolder title fix is in.

Avatar
martimiz

Forum Moderator, 1132 Posts

8 September 2009 at 11:11pm

@silk78

>> I found an entry in apache error-log, though:

>> File does not exist: /srv/rsv-kanu_neu/bulk_uploader

Don't know if you found out how to get reid of this log-entry yet. The only reference I found: in dataobject_manager/javascript/facebox.js - line 100: replace

<img src="/bulk_uploader/images/closelabel.gif" title="close" class="close_image" /> \

by

<img src="/dataobject_manager/images/closelabel.gif" title="close" class="close_image" /> \

cheers

Avatar
UncleCheese

Forum Moderator, 4096 Posts

9 September 2009 at 2:02am

Edited: 09/09/2009 2:03am

My lord.. The bulk_uploader module hasn't existed in many, many months. No idea why that was in there.