Messages

46

Older versions / "Error: The file does not exists" Contact import failure

« on: January 22, 2010, 06:51:25 pm »

Hello,

I wanted to share that I was having a similar issue to the one in this thread:
http://forums.fengoffice.com/index.php?topic=1445.0
"unable to import contacts"

I was doing this from my Windows 7 machine, and tried opening up the NTFS permissions on the file wide open. I also tried running my browser as admin. I edited ./fengoffice/php.ini to look like:

Code: [Select]

max_execution_time = 300
post_max_size = 10M
upload_max_filesize = 10M
memory_limit = 64M...to try and see if it was failing during upload. I also temporarily set ./fenoffice/upload/ to permissions 777.

None of these things helped. What fixed it was to cut my import file into two pieces. It turns out that my file was 2.23mb and anything under ~2mb would proceed fine. Anything bigger would fail with the error "The file does not exists."

I hope this is helpful.

Thank you,
Mark

fengoffice 1.6.1
ubuntu server 8.04 LTS x86

47

Development / Re: LDAP Integration

« on: January 19, 2010, 09:29:03 pm »

It seems that there is a bug in the search functions.
I have corrected a bit the code and it works :
In the function
       function isValidPasswordLdap($user, $password, $config) {
In the User.class.pho file the end of the function is now :

                if ($search->count() != 1) {
                    return false;
                }
                foreach($search as $dn => $dummy) {
                }
                $bind_result = $ldap->bind( $dn, $password);

                if (PEAR::isError($bind_result)) {
                        return false;
                }
                return true;
}

I see what you've done with the test for the admin user (thanks for that!), but I don't understand what necessitated the patch to the search function? Accessing the data with a hard coded zero index should be okay since there was a prior test for there being only one returned result from ldap.

My setup is working without this search patch so I'm curious as to the logic.

Thank you,
Mark