All right, I created the Jira (http://jira.magnolia-cms.com/browse/MGNLDATA-81). If I can spend time on it, I'll submit a patch.
Regards, Bert > -----Original Message----- > From: dev-list-ow...@magnolia-cms.com [mailto:dev-list-ow...@magnolia- > cms.com] On Behalf Of Grégory Joseph > Sent: dinsdag 27 oktober 2009 17:42 > To: Magnolia Dev-List > Subject: Re: [magnolia-dev] RE: datamodule dialogs do not check > required fields > > > Jira.. and a patch ;) > > I don't have the code right under my eyes right now, but it sounds > like the data module uses its own dialog class, which somehow misses > some of the functionality... if you see where this happens in > DialogControlImpl, I suppose you could easily patch it ;) > > -g > > > On Oct 27, 2009, at 3:48 PM, Bert Leunis wrote: > > > Hello Basel, > > > > Can anyone help with this matter? Can I quick-fix this somewhere? I > > can add this problem to Jira if you want. > > > > Autumn greetings, Bert > > > > From: dev-list-ow...@magnolia-cms.com [mailto:dev-list- > ow...@magnolia-cms.com > > ] On Behalf Of Bert Leunis > > Sent: donderdag 15 oktober 2009 9:34 > > To: Magnolia Dev-List > > Subject: [magnolia-dev] datamodule dialogs do not check required > > fields > > > > Dear Magnolians, > > > > On regular dialogs (for paragraphs for the website) you can set a > > "required" node with value "true". The label of that field is then > > marked with a *. The check on that is done in the class > > info.magnolia.cms.gui.dialog.DialogControlImpl. > > > > We discovered that this functionality does not work on dialogs in > > the datamodule. The DialogControlImpl class does not play a role > > there. Is there an easy way to implement this behaviour for the > > datamodule dialogs? > > > > Best of regards, Bert > > > > > > > > ---------------------------------------------------------------- > > For list details see > > http://www.magnolia-cms.com/home/community/mailing-lists.html > > To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com> > > ---------------------------------------------------------------- > > > > > > ---------------------------------------------------------------- > > For list details see > > http://www.magnolia-cms.com/home/community/mailing-lists.html > > To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com> > > ---------------------------------------------------------------- > > > ---------------------------------------------------------------- > For list details see > http://www.magnolia-cms.com/home/community/mailing-lists.html > To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com> > ---------------------------------------------------------------- ---------------------------------------------------------------- For list details see http://www.magnolia-cms.com/home/community/mailing-lists.html To unsubscribe, E-mail to: <dev-list-unsubscr...@magnolia-cms.com> ----------------------------------------------------------------
