Changeset 1534 in subversion

Timestamp:

Jun 14, 2008 1:35:26 PM (5 years ago)

Author:

thomasb

Message:

Complete implementation of #1484601: add link for saving sender to address book and reload message

Location:

trunk/roundcubemail

Files:

• program/include/rcube_message.php (modified) (2 diffs)
• program/js/app.js (modified) (2 diffs)
• program/localization/de_CH/labels.inc (modified) (1 diff)
• program/localization/de_DE/labels.inc (modified) (1 diff)
• program/localization/en_US/labels.inc (modified) (1 diff)
• program/steps/mail/show.inc (modified) (3 diffs)
• skins/default/mail.css (modified) (1 diff)

trunk/roundcubemail/program/include/rcube_message.php

@@ -43 +43 @@
   public $attachments = array();
   public $subject = '';
+  public $sender = null;
   public $is_safe = false;
   
@@ -54 +55 @@
     $this->headers = $this->imap->get_headers($uid);
     $this->subject = rcube_imap::decode_mime_string($this->headers->subject, $this->headers->charset);
+    list(, $this->sender) = each($this->imap->decode_address_list($this->headers->from));
     
-    $this->is_safe = (intval($_GET['_safe']) || $_SESSION['safe_messages'][$uid]) ? true : false;
-    $_SESSION['safe_messages'][$uid] = $this->is_safe;
+    $this->set_safe((intval($_GET['_safe']) || $_SESSION['safe_messages'][$uid]));
+    $this->set_safe(0);
     
     $this->opt = array(

trunk/roundcubemail/program/js/app.js

@@ -186 +186 @@
           if (this.gui_objects.remoteobjectsmsg)
             this.gui_objects.remoteobjectsmsg.style.display = 'block';
-          this.enable_command('load-images', true);
+          this.enable_command('load-images', 'always-load', true);
           }
 
@@ -705 +705 @@
         this.mark_message(flag, uid);
         break;
+        
+      case 'always-load':
+        if (this.env.uid && this.env.sender) {
+          this.add_contact(urlencode(this.env.sender));
+          window.setTimeout(function(){ ref.command('load-images'); }, 300);
+          break;
+        }
         
       case 'load-images':

trunk/roundcubemail/program/localization/de_CH/labels.inc

@@ -185 +185 @@
 $labels['nosubject']  = '(kein Betreff)';
 $labels['showimages'] = 'Bilder anzeigen';
+$labels['alwaysshow'] = 'Bilder von $sender immer zeigen';
 
 $labels['htmltoggle'] = 'HTML';

trunk/roundcubemail/program/localization/de_DE/labels.inc

@@ -147 +147 @@
 $labels['nosubject'] = '(kein Betreff)';
 $labels['showimages'] = 'Bilder anzeigen';
+$labels['alwaysshow'] = 'Bilder von $sender immer zeigen';
 $labels['htmltoggle'] = 'HTML';
 $labels['plaintoggle'] = 'Nur-Text';

trunk/roundcubemail/program/localization/en_US/labels.inc

@@ -185 +185 @@
 $labels['nosubject']  = '(no subject)';
 $labels['showimages'] = 'Display images';
+$labels['alwaysshow'] = 'Always show images from $sender';
 
 $labels['htmltoggle'] = 'HTML';

trunk/roundcubemail/program/steps/mail/show.inc

@@ -48 +48 @@
   // check known senders to display images
   if (!$MESSAGE->is_safe 
-        && !$_SESSION['safe_messages'][$MESSAGE->uid]
-        && !isset($_GET['_safe']) 
-        && $RCMAIL->config->get('addrbook_show_images')
-        && $MESSAGE->has_html_part())
-    {
-      $CONTACTS = new rcube_contacts($DB, $_SESSION['user_id']);
-      $senders = $IMAP->decode_address_list($MESSAGE->headers->from);
-
-      foreach ($senders as $sender)
-        if ($sender['mailto'] && $CONTACTS->search('email', $sender['mailto'], true, false)->count)
-          {
-          $MESSAGE->set_safe();
-          break;
-          }
-    }
+      && !empty($MESSAGE->sender['mailto'])
+      && $RCMAIL->config->get('addrbook_show_images')
+      && $MESSAGE->has_html_part()) {
+    $CONTACTS = new rcube_contacts($DB, $_SESSION['user_id']);
+    
+    if ($CONTACTS->search('email', $MESSAGE->sender['mailto'], true, false)->count) {
+      $MESSAGE->set_safe(true);
+    }
+  }
 
   // allow caching, unless remote images are present
@@ -86 +80 @@
   $OUTPUT->set_env('uid', $MESSAGE->uid);
   $OUTPUT->set_env('safemode', $MESSAGE->is_safe);
+  $OUTPUT->set_env('sender', $MESSAGE->sender['string']);
   
   // check for unset disposition notification
@@ -175 +170 @@
 
 function rcmail_remote_objects_msg($attrib)
-  {
-  global $CONFIG, $OUTPUT;
+{
+  global $MESSAGE, $RCMAIL;
   
   if (!$attrib['id'])
     $attrib['id'] = 'rcmremoteobjmsg';
-
-  // allow the following attributes to be added to the <div> tag
-  $attrib_str = create_attrib_string($attrib, array('style', 'class', 'id'));
-  $out = '<div' . $attrib_str . ">";
-  
-  $out .= sprintf('%s&nbsp;<a href="#loadimages" onclick="%s.command(\'load-images\')">%s</a>',
-                  Q(rcube_label('blockedimages')),
-                  JS_OBJECT_NAME,
-                  Q(rcube_label('showimages')));
-  
-  $out .= '</div>';
-  
-  $OUTPUT->add_gui_object('remoteobjectsmsg', $attrib['id']);
-  return $out;
-  }
+  
+  $msg = Q(rcube_label('blockedimages')) . '&nbsp;';
+  $msg .= html::a(array('href' => "#loadimages", 'onclick' => JS_OBJECT_NAME.".command('load-images')"), Q(rcube_label('showimages')));
+  
+  // add link to save sender in addressbook and reload message
+  if ($MESSAGE->sender['mailto'] && $RCMAIL->config->get('addrbook_show_images')) {
+    $msg .= ' ' . html::a(array('href' => "#alwaysload", 'onclick' => JS_OBJECT_NAME.".command('always-load')", 'style' => "white-space:nowrap"),
+      Q(rcube_label(array('name' => 'alwaysshow', 'vars' => array('sender' => $MESSAGE->sender['mailto'])))));
+  }
+  
+  $RCMAIL->output->add_gui_object('remoteobjectsmsg', $attrib['id']);
+  return html::div($attrib, $msg);
+}
 
 

trunk/roundcubemail/skins/default/mail.css

@@ -731 +731 @@
 {
   display: none;
-  height: 20px;
+  margin: 8px;
   min-height: 20px;
-  margin: 8px 8px 0px 8px;
-  padding: 10px 10px 6px 46px;  
+  padding: 10px 10px 6px 46px;
 }