[imp] Fetchmail multiple drivers/preferences

Rick Romero rick at havokmon.com
Sun Dec 7 19:21:17 UTC 2008


Hey guys,

I've got a short little patch for handling multiple fetchmail drivers.  
Currently, if you create a new driver in imp/lib/Fetchmail/drivers, it 
becomes available but uses the 'fetchmailprefs' template - which may not 
wholly apply.  This patch changes the template file to $driver . 
"prefs.html" so you don't have to confuse your users :)

I realize the template driver has a bit more 'oomph' to it, like 
selecting which account to use or creating a new one.  Maybe that should 
be split out into a separate file?   This patch will break loading the 
template for creating or editing.  In that case, it should be named 
'imapprefs.html', as 'imap' is the driver name that comes with IMP by 
default. 

Rick

--- /usr/local/www/data/horde/imp/fetchmailprefs.php    Fri Jul 25 
15:45:49 2008
+++ /usr/local/www/sub/beta/horde/imp/fetchmailprefs.php        Sun Dec  
7 13:17:29 2008
@@ -195,7 +195,11 @@
 Prefs_UI::generateHeader(null, $chunk);
 Horde::addScriptFile('prototype.js', 'imp', true);
 Horde::addScriptFile('fetchmailprefs.js', 'imp', true);
-echo $t->fetch(IMP_TEMPLATES . '/fetchmail/fetchmailprefs.html');
+if (!$driver){
+   echo $t->fetch(IMP_TEMPLATES . '/fetchmail/fetchmailprefs.html');
+}else{
+   echo $t->fetch(IMP_TEMPLATES . '/fetchmail/'.$driver.'prefs.html');
+}
 if (!$chunk) {
     require $registry->get('templates', 'horde') . '/common-footer.inc';
 }



More information about the imp mailing list