Ticket #883: new_gmail_v3.patch

src/notify.py

@@ -195 +195 @@
                         else:
                                 txt = ''
                 elif event_type == _('New E-mail'):
-                        txt = _('You have new E-mail on %s.') % (jid)
+                        no_of_messages = int(msg_type)
+                        text = i18n.ngettext('You have %d new E-mail message', 'You have %d new E-mail messages', no_of_messages, no_of_messages, no_of_messages)
+                        txt = _('%(new_mail_gajim_ui_msg)s on %(gmail_mail_address)s') % {'new_mail_gajim_ui_msg': text, 'gmail_mail_address': jid}
                         ntype = 'gmail.notify'
                         img = 'single_msg_recv.png' #FIXME: find a better image
+                        self.msg_type = msg_type = 'gmail'
                 else:
                         # defaul failsafe values
                         img = 'chat_msg_recv.png' # img to display

src/dialogs.py

@@ -924 +924 @@
                         dodgerblue = gtk.gdk.color_parse('dodgerblue')
                         close_button.modify_bg(gtk.STATE_NORMAL, dodgerblue)
                         eventbox.modify_bg(gtk.STATE_NORMAL, dodgerblue)
-                        txt = _('You have new E-mail on %s.') % (jid)
+                        no_of_messages = int(msg_type)
+                        text = i18n.ngettext('You have %d new E-mail message', 'You have %d new E-mail messages', no_of_messages, no_of_messages, no_of_messages)
+                        txt = _('%(new_mail_gajim_ui_msg)s on %(gmail_mail_address)s') % {'new_mail_gajim_ui_msg': text, 'gmail_mail_address': jid}
+                        self.msg_type = msg_type = 'gmail'
                         event_description_label.set_markup('<span foreground="black">%s</span>' % txt)
                 # position the window to bottom-right of screen
                 window_width, self.window_height = self.window.get_size()

src/gajim.py

@@ -852 +852 @@
                         notify.notify(_('File Transfer Error'),
                                 jid, account, 'file-send-error', file_props)
                                 
-        def handle_event_gmail_notify(self, account, jid):
+        def handle_event_gmail_notify(self, account, array):
+                jid = array[0]
+                newmsgs = array[1]
                 if gajim.config.get('notify_on_new_gmail_email'):
-                        notify.notify(_('New E-mail'), jid, account)
+                        notify.notify(_('New E-mail'), jid, account, newmsgs)
 
         def add_event(self, account, jid, typ, args):
                 '''add an event to the awaiting_events var'''
@@ -1322 +1324 @@
                         ev = gajim.get_first_event(account, jid, typ)
                         # Open the window
                         self.roster.open_event(account, jid, ev)
+                elif typ == message_control.TYPE_GMAIL:
+                        if gajim.config.get_per('accounts', account, 'savepass'):
+                                url = ('http://www.google.com/accounts/ServiceLoginAuth?service=mail&Email=%s&Passwd=%s&continue=https://mail.google.com/mail') % (gajim.config.get_per('accounts', account, 'name'),gajim.config.get_per('accounts', account, 'password'))
+                        else:
+                                url = ('http://mail.google.com/')
+                        helpers.launch_browser_mailer('url', url)
                 if w:
                         w.set_active_tab(jid)
                         w.window.present()

src/common/connection.py

@@ -331 +331 @@
                         else:
                                 self.dispatch('VCARD', vcard)
 
-        def _gMailCB(self, con, gm):
+        def _gMailNewMailCB(self, con, gm):
                 """Called when we get notified of new mail messages in gmail account"""
                 if not gm.getTag('new-mail'):
                         return
                 if gm.getTag('new-mail').getNamespace() == common.xmpp.NS_GMAILNOTIFY:
+                        # we'll now ask the server for the exact number of new messages
                         jid = gajim.get_jid_from_account(self.name)
-                        gajim.log.debug(('Notifying user of new gmail e-mail on %s.') % (jid))
-                        self.dispatch('GMAIL_NOTIFY', jid)
+                        gajim.log.debug(('Got notification of new gmail e-mail on %s. Asking the server for more info.') % (jid))
+                        iq = common.xmpp.Iq(typ = 'get')
+                        iq.setAttr('id', '13')
+                        query = iq.setTag('query')
+                        query.setNamespace(common.xmpp.NS_GMAILNOTIFY)
+                        self.to_be_sent.append(iq)
                         raise common.xmpp.NodeProcessed
+                
+        def _gMailQueryCB(self, con, gm):
+                """Called when we receive results from Querying the server for mail messages in gmail account"""
+                if not gm.getTag('mailbox'):
+                        return
+                if gm.getTag('mailbox').getNamespace() == common.xmpp.NS_GMAILNOTIFY:
+                        newmsgs = gm.getTag('mailbox').getAttr('total-matched')
+                        if newmsgs != '0':
+                                # there are new messages
+                                jid = gajim.get_jid_from_account(self.name)
+                                gajim.log.debug(('User has %s new gmail e-mails on %s.') % (newmsgs, jid))
+                                self.dispatch('GMAIL_NOTIFY', (jid, newmsgs))
+                        raise common.xmpp.NodeProcessed
 
         def _messageCB(self, con, msg):
                 """Called when we receive a message"""
@@ -1746 +1764 @@
                         common.xmpp.NS_PRIVATE)
                 con.RegisterHandler('iq', self._HttpAuthCB, 'get',
                         common.xmpp.NS_HTTP_AUTH)
-                con.RegisterHandler('iq', self._gMailCB, 'set',
+                con.RegisterHandler('iq', self._gMailNewMailCB, 'set',
                         common.xmpp.NS_GMAILNOTIFY)
+                con.RegisterHandler('iq', self._gMailQueryCB, 'result',
+                        common.xmpp.NS_GMAILNOTIFY)
                 con.RegisterHandler('iq', self._ErrorCB, 'error')
                 con.RegisterHandler('iq', self._IqCB)
                 con.RegisterHandler('iq', self._StanzaArrivedCB)

src/message_control.py

@@ -24 +24 @@
 TYPE_CHAT = 'chat'
 TYPE_GC = 'gc'
 TYPE_PM = 'pm'
+TYPE_GMAIL = 'gmail'
 
 ####################
 # FIXME: Can't this stuff happen once?