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   52973 Views

Avatar
UncleCheese

Forum Moderator, 4096 Posts

28 July 2009 at 3:48am

Could you post a link to the page that is having this problem?

Avatar
Evan

Community Member, 2 Posts

3 August 2009 at 6:57pm

Edited: 03/08/2009 6:57pm

Hi, I'm having a small problem with the Image Gallery module. I've got a new Page type which extends ImageGalleryPage, however it doesn't do anything more than redefine some of the presets. When creating a new instance of this page, it seems the title of the Folder that is being created by Image Gallery is not properly inheriting the title of the Page. The behaviour is a little strange as the title in the left-hand tree view is correct, as is the title field in the main view, however the title in the button and in the table below is still displaying the default value ('New ProjectPage' in my case). I've got a screenshot of this [url=http://img190.imageshack.us/img190/356/imagegalleryj.gif]here[/url].

Hitting the 'Save folder name' button updates all the values as expected, however we'd rather avoid having clients perform this extra step.

ImageGallery version: Stable
Browser/OS: FF3/Windows 7

Avatar
t|m

Community Member, 19 Posts

5 August 2009 at 3:38pm

Edited: 05/08/2009 4:03pm

when using DOM in AssetAdmin it doesnt load the fields from the specific file Dataobject.
instead it uses hardcoded form fields.

(using latest trunk r220)

(not handy when using Decorators :(... )

a step towards getting it dynamic would be using
singleton($sourceClass)->getCMSFields()

and adding/removing the required formfields using the FieldSet::remove,replace etc methods...

Avatar
UncleCheese

Forum Moderator, 4096 Posts

6 August 2009 at 1:57am

That's a good idea. I'll try to get it in today. Thank you.

Avatar
t|m

Community Member, 19 Posts

6 August 2009 at 12:50pm

just made some changes in order to have the childfields auto generated:

AssetManager
http://php.pastebin.com/d6c97c875

AssetManagerFolder
http://php.pastebin.com/d4f4d4b50

cheers

Avatar
martimiz

Forum Moderator, 1132 Posts

12 August 2009 at 11:54pm

hi UncleCheese
I honestly don't know if these things were mentioned before (rev221), anyway:

1. When adding a new album, it seems that the popup's album title field is not properly escaped: entering a name like foto's will generate a mysql error. This only happens at first save, not when updating

2. Adding a new image using 'import from an existing folder' and then removing it, also removes it from other galleries that use it, without actually removing the record from these galleries, so emty spaces are shown on the gallery page. It seems the existing links were changed to point to the new imagelocation even though the 'old' images do still exitst. Clicking such an 'empty image' in the backend will result in an error:

Trying to get property of non-object
GET /admin/EditForm/field/GalleryItems/item/7/edit

Line 113 in /home/WWW/fgf.balbus.tk/www/image_gallery/code/ImageGalleryManager.php

Avatar
abes

Community Member, 11 Posts

13 August 2009 at 4:32am

Edited: 13/08/2009 4:32am

Hi!

There is a problem with the SimpleHTMLEditorField, which is not visible in some circumstances:
I figured out, that this failure only happens when there is no DatePickerField in the same form. If one puts a DatePickerField into the form, all SimpleHTMLEditorFields appear. After some debugging I worked out, that missing requirements are the reason for the problem. Following solution works fine for me:

I just added the two JQuery-Requirements from DatePickerField::Field() to the top of SimpleHTMLEditorField::FieldHolder().

   public function FieldHolder()
   {
      Requirements::javascript("jsparty/jquery/jquery.js");
      Requirements::javascript("dataobject_manager/javascript/jquery-ui.1.7.js");
      
      [ ... ]
      
   }
}

This bug is OS and browser independent. Off course I use latest trunk R221.

Avatar
UncleCheese

Forum Moderator, 4096 Posts

13 August 2009 at 7:31am

Thanks. This class is going to be deprecated soon by yet another minimal html editor field called "SimpleTinyMCEField" that will use the jQuery version of TinyMCE, and that way we don't have to drive ourselves crazy supporting all kinds of third party tools.