SiT! Bugs

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001739SiT!LDAPpublic2011-12-01 09:032012-01-24 13:30
Reporterv0hkus 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusconfirmedResolutionopen 
PlatformLinuxOSLinuxOS Version2.6+later
Product Version3.65 LTS 
Target VersionFixed in Version 
Summary0001739: "LDAP Customer default site" must be the required field if the "LDAP Customer Group" is specified
Description"LDAP Customer default site" must be the required field if the "LDAP Customer Group" is specified, because of the function check_valid in classes.inc.php (line 315). If the default site is not specified while Customer contacts are on the LDAP server, the ldapSync action will fail and no error message will be displayed.
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
User avatar (0004345)
ivan (administrator)
2011-12-03 10:21

As described this is not a 'block' bug, please see http://sitracker.org/wiki/Bugs#A_guide_to_bug_severity [^]
User avatar (0004352)
paulh (administrator)
2011-12-04 20:19

HI,

Whilst this field isn't marked as required (so yes there is a bug) its impossible not to select one or are you seeing that its possible to select no site?

As an as side the code that inserts the record performed a validity check and one of these is that the site must be set and errors should be triggered.

Cheers
(0004354)
v0hkus (reporter)
2011-12-05 12:00

This field is left blank if there is no Customer sites at all. My first attempt to test LDAP sync was before I've added first site, so it was not possible to select one on the LDAP Configuration page.

I think a field validation before saving changes on LDAP Configuration page is a good idea.
User avatar (0004430)
ivan (administrator)
2012-01-24 13:30

Validation of this type is currently hard to do with our current configuration page code.

- Issue History
Date Modified Username Field Change
2011-12-01 09:03 v0hkus New Issue
2011-12-03 10:21 ivan Note Added: 0004345
2011-12-03 10:21 ivan Severity block => minor
2011-12-04 20:19 paulh Note Added: 0004352
2011-12-04 20:19 paulh Status new => feedback
2011-12-05 12:00 v0hkus Note Added: 0004354
2011-12-05 12:00 v0hkus Status feedback => new
2012-01-24 13:30 ivan Note Added: 0004430
2012-01-24 13:30 ivan Status new => confirmed


Copyright © 2000 - 2019 MantisBT Team
Powered by Mantis Bugtracker