http://lists.horde.org/archives/cvs/Week-of-Mon-20051205/051894.html



This code was needed as the default parameters were potentially not 
being set correctly from the information in the backends.php file.

I just pulled the CVS version and yes, it does seem to fix the problem.

Care to elaborate about the right way to fix this? Where were these 
changes committed? in redirect.php?

No, this isn't right.  Instead, try what I just committed to CVS.

Sorry, minutes after the last post I might have found a fix (context 
diff attached).

In ./lib/Session.php:



- add 'maildomain' to the loop:

foreach (array('port', 'protocol', 'smtphost', 'smtpport') as $param) {

- remove the line a bit further down:

$_SESSION['mimp']['maildomain'] = $args['maildomain'];



That did it for me.

Well, my problem seems to revolve around $_SESSION['mimp']['maildomain']

not being set. If I force it to a hardcoded value (the same value I 
have set in servers.php) in redirect.php like:

     $sessArray = array('server'     => Util::getFormData('server'),

                        'realm'      => Util::getFormData('realm'),

                        'port'       => Util::getFormData('port'),

                        'protocol'   => Util::getFormData('protocol'),

                        'maildomain' => Util::getFormData('maildomain'));

change last line to:

                        'maildomain' => 'ait.ac.th');

the problem goes away. Somehow this value isn't set to the value found 
in the corresponding (and only) entry of ./config/servers.php:

$servers['topmail'] = array(

...

'maildomain' => 'ait.ac.th',

...

My conf.php has:

$conf['server']['server_list'] = 'none';

$conf['server']['change_server'] = false;

$conf['server']['change_port'] = false;

$conf['server']['change_protocol'] = false;

$conf['server']['change_folders'] = false;

and I guess that's relevant to this problem.

I still can't figure out yet where it should be set and isn't: in 
./redirect.php? in ./lib/Session.php (MIMP_Session::createSession)? in 
./templates/login/login.inc (just a form entry)?

I'll give this another try next week. Any input is welcome.

Are you talking about the problem I initially reported (when composing 
new mail) or what

Chuck has seen (when replying to a mail sent to a non-default identity)?

I've tried to apply your suggested change but that didn't fix my problem.

This statement appears at line #123 in my version (1.0-alpha) and not 
#129, by the way.

* $Horde: mimp/compose.php,v 1.66 2005/10/09 17:57:10 slusarz Exp $

I really need to find some time to dig into this myself.

I'm not seeing the reported behavior, but when I reply to a message 
that was sent to a non-default Identity, the compose window defaults 
to the default identity, not the one it should (and does in IMP). 
Looks like this is related.



Michael, I'm not quite following things right now (not really time), 
but it looks like MIMP's compose.php expects to be passed an 
'identity' form parameter, but MIMP doesn't set that in its compose 
links, whereas IMP does...

We have experienced the same behaviour.



In compose.php, in the function _formatAddr($addr), in  switch   
($actionID) the call at line 127    $_GET['identity'] = $default; does 
not  pick up the maildomain.



Changing this to the function call used by compose.php in imp works:   
$_GET['identity'] = $identity->getMatchingIdentity($fromaddr);

Horde-3.0.5, Mimp-h3-1.0-alpha



When users select 'Compose', the From: field comes up

as "user@" (no domain).



This doesn't happen if that user has set up an explicit From: address

in his default identity (in that case the From: comes up OK).



In MIMP's server.php, 'maildomain' is set correctly (see attached server.php)



The same setting works in IMP.