Opened 13 years ago

Last modified 13 years ago

#417 new defect

System configuration page validation is over-zealous.

Reported by: justinrowles Owned by: geonetwork-devel@…
Priority: major Milestone: Future release
Component: General Version: v2.6.2
Keywords: Cc:

Description

I can't save my settings because the LDAP values are not filled - but I have not chosen LDAP authentication.

Attachments (1)

trac417.view.js.diff (2.7 KB ) - added by justinrowles 13 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 by josegar74, 13 years ago

Can you provide more information about the error?

I can save settings without problem. I enabled INSPIRE setting and checked the search form shows INSPIRE panel correctly. Also no error when saving the setting.

Using default 2.6.2 installer with jetty.

Maybe also can try to clean cache and cookies in browser so being sure it's not using a cached old version of js files.

comment:2 by justinrowles, 13 years ago

If you check the box for LDAP (which expands it), by default it has dummy values in it. For some reason, I don't have those values (I don't think I deleted them). I suspect your save works because the dummy values are still present.

The bug is that I don't have LDAP authentification checked when I save and those fields are still checked.

Attached are the changes to view.js so that the front end behaves correctly (using the 'precond' element of the rules, but the back end still won't allow you to save a null uid (for example) because of the mandatory=true in org.fao.geonet.services.config.Set.java at line 159.

by justinrowles, 13 years ago

Attachment: trac417.view.js.diff added

comment:3 by justinrowles, 13 years ago

NB this patch shouldn't be used without addressing the back end issue too.

comment:4 by fxp, 13 years ago

Milestone: v2.6.3Future release
Note: See TracTickets for help on using tickets.