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

ImageDataObjectManager showing all photos on every page


4 Posts   1651 Views


8 February 2010 at 4:44pm Community Member, 190 Posts

I have an odd gallery implementation, so I've decided to use the ImageDataObjectManager.

I've added it to Page.php so I could have a separate gallery for each page. It seems to be working when I look at the CMS (it's only showing the images I've uploaded for that specific page). But when viewing the site, every page is outputting all photos that were added no matter what page they were uploaded to.



class GalleryPhoto extends DataObject {
   static $db = array (
      'Category' => 'Varchar',
      'PhotoCredit' => 'Varchar(200)'

   static $has_one = array (
      'Attachment' => 'File',
      'Page' => 'Page'

   public function getCMSFields_forPopup()
      return new FieldSet(
         new TextField('Category'),
         new TextField('PhotoCredit'),
         new FileIFrameField('Attachment')



static $has_many = array (
   'PhotoGallery' => 'GalleryPhoto'
function Lightbox() {      
   return DataObject::get('GalleryPhoto');

<% control Lightbox %>
<% end_control %>


9 February 2010 at 1:45am Community Member, 288 Posts

The problem is your Lightbox getter method.
Since you're doing return DataObject::get('GalleryPhoto'); without any where clause you will get all the gallery photos back.
If all you want is to get the Gallery photos related to the current page then you don't need to write your own method.

Just write this in your template instead.

<% control PhotoGallery %>
<% end_control %>


9 February 2010 at 3:12am 4085 Posts

If you're going to use a DOM to manage relationships on subclasses, you need to set its parent class. Think about it -- you're passing $this as the controller, so on the class PageSubclass, the DOM will try to set the PageSubclassID for that object, when in fact, it should be a PageID.



10 February 2010 at 7:54am Community Member, 190 Posts

I first tried using the source name (PhotoGallery) instead of the source class, and that seemed to do the trick.

Thank you both!