[Tickets #14610] 'attach_body_check' hook prevents IMP smartmobile view to send message with 'attachment' word when there's no attachment

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: https://bugs.horde.org/ticket/14610
------------------------------------------------------------------------------
  Ticket             | 14610
  Created By         | leonardo at cefetmg.br
  Summary            | 'attach_body_check' hook prevents IMP smartmobile view
                     | to send message with 'attachment' word when there's no
                     | attachment
  Queue              | IMP
  Version            | 6.2.17
  Type               | Bug
  State              | Unconfirmed
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


leonardo at cefetmg.br (2017-04-13 17:09) wrote:

Scenario: User is composing a message on the IMP smartmobile view. The  
'attach_body_check' IMP hook is enabled and configured with a set of  
'attachments' keywords. Horde/IMP is installed using Debian 8 official  
packages, Horde is 5.2.1 and IMP is 6.2.2.

It's well know that Debian ships a jQuery version which is not IMP  
compatible. So, to be able to use even the very basic mail features in  
smartmobile view, I needed to

jQuery / jQuery-mobile libs was update to versions 1.12.4 / 1.3.2,  
this done just by sym-linking default files to the updated ones. Horde  
smartmobile view worked correctly (not only IMP, but Turba, Kronolith  
and Ingo also).

Symptoms: When using the IMP smartmobile view, it will not be possible  
to send a message without attachments if in the message body text  
there is any of the keywords defined in the 'attach_body_check' IMP  
hook.

The first attempt to send will show -- as it is expected -- a warning  
message informing that none attachments are indeed present. However,  
the next attempts present the same behavior: a message warns about the  
absence of attachments and the message is not sent. It will only be  
sent if: 1) you remove the keyword(s) from the message, or 2) add some  
attachment, or 3) disable the hook.

Remarks:
1. Only the smartmobile view is affected. Dynamic view on both desktop  
and mobile main browsers sends the messages at the 2nd attempt.
2. The tests where performed with the default Horde and IMP prefs. The  
only hook enabled was 'attach_body_check'.
3. This issue also affects IMP 6.2.17 which is in the 'php-horde-imp'  
Debian stretch package.