Ticket #417 (new defect)

Opened 2 years ago

Last modified 2 years ago

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

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

Change History

Changed 2 years ago by josegar74

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.

Changed 2 years ago by justinrowles

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.

Changed 2 years ago by justinrowles

Changed 2 years ago by justinrowles

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

Changed 2 years ago by fxp

  • milestone changed from v2.6.3 to Future release
Note: See TracTickets for help on using tickets.