[ postfixadmin-Patches-2925620 ] password column name in configuration

[SourceForge.net <no...@so...>]

Patches item #2925620, was opened at 2010-01-04 14:10
Message generated for change (Comment added) made by christian_boltz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=937966&aid=2925620&group_id=191583

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Database
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Gabriel Serme (eirmag)
Assigned to: Nobody/Anonymous (nobody)
Summary: password column name in configuration

Initial Comment:
Sometimes you may want to have multiple columns in your database to store your password, or simply have a different name than "password". The problem is that the column name is hard written in source code.

This patch gives you the possibility to change the database password column name for USERS ONLY from the configuration file. Postfixadmin's column password remains "password".

The new config name is : "database_password_column_name", and impacted files are config.inc.php and model/UserHandler.php

Greetings for this new year, 
Gabriel

----------------------------------------------------------------------

>Comment By: Christian Boltz (christian_boltz)
Date: 2010-01-05 13:19

Message:
Can you please describe _why_ the additional or renamed password column is
useful? At the moment I don't see the advantage. OTOH your patch would make
upgrades harder because database changes would be applied to the wrong
column etc.

I'm not totally against merging this patch, but I'd like to know your
usecase and reason first. (Maybe we find another solution for your problem,
which would be the better way IMHO.)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=937966&aid=2925620&group_id=191583