[FTPSERVER-450] PropertiesUserManager userDataFile is always NULL.

• Type: Bug
• Status: Open
• Priority: Blocker
• Resolution: Unresolved
• Affects Version/s: 1.0.6
• Component/s: Core
• Environment: JDK 6; Windows 7 64 BIT
• Assignee: Unassigned
• Reporter: Aizaz AZ
• Created: 16/Oct/13 09:09
• Updated: 21/Oct/13 09:43

Description

PropertiesUserManager userDataFile is always NULL. Because you didn't set your class variable value equal to what you get in constructor when PropertiesUserManagerFactory calls your constructor new PropertiesUserManager(passwordEncryptor, userDataFile, adminName); of PropertiesUserManager,just need to do below super this.userDataFile = userDataFile; and issue is fixed

                PropertiesUserManagerFactory propUserManagerFactory = new PropertiesUserManagerFactory();
		propUserManagerFactory.setFile(new File("ftpusers.properties"));
		propUserManagerFactory.setPasswordEncryptor(new ClearTextPasswordEncryptor());
		mUserManager = (PropertiesUserManager) propUserManagerFactory.createUserManager();
		mFTPServerFactory.setUserManager(mUserManager);

Set file has no effect because factory will call your constructor

PropertiesUserManager(passwordEncryptor, userDataFile,
                    adminName)
and you didn't set this.userDataFile = userDataFile;

complete

  public PropertiesUserManager(PasswordEncryptor passwordEncryptor,
            File userDataFile, String adminName) {
        super(adminName, passwordEncryptor);
        
        this.userDataFile = userDataFile;
        
        loadFromFile(userDataFile);
    }

Issue fixed

Note: Please find attached class with issue fixed