6.0.0-beta6
3/30/26

[#14090] gpg mail display broken
Summary gpg mail display broken
Queue Horde Framework Packages
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester o+horde (at) immerda (dot) ch
Created 8/20/15 (3875 days ago)
Due
Updated 8/20/15 (3875 days ago)
Assigned 8/20/15 (3875 days ago)
Resolved 8/20/15 (3875 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
08/20/2015 11:27:39 PM Michael Rubinsky Version ⇒
Queue ⇒ Horde Framework Packages
 
08/20/2015 11:25:29 PM Michael Rubinsky Taken from Michael Slusarz
State ⇒ Resolved
 
08/20/2015 09:39:16 PM o+horde (at) immerda (dot) ch Comment #9 Reply to this comment
Can you verify that this fixes it for you?
yes it does, thanks
08/20/2015 04:24:55 PM Michael Rubinsky Comment #8
State ⇒ Feedback
Reply to this comment
Can you verify that this fixes it for you?
08/20/2015 04:24:22 PM Git Commit Comment #7 Reply to this comment
Changes have been made in Git (master):

commit 6f245104fbfc02bceb5707c31600fffe183ab058
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Thu Aug 20 12:22:52 2015 -0400

     Bug: 14090  Closures do not have full access to the $this scope 
prior to 5.4

     While we can give it access to $this by assigning it to another
     variable, that variable does not have access to non-public members.

  .../Crypt/lib/Horde/Crypt/Pgp/Backend/Binary.php   |    6 +++++-
  1 files changed, 5 insertions(+), 1 deletions(-)

http://github.com/horde/horde/commit/6f245104fbfc02bceb5707c31600fffe183ab058
08/20/2015 04:19:10 PM Michael Rubinsky Comment #6 Reply to this comment
To clarify, the issue isn't with the $this directly, since we work 
around that by assigning $this to another variable that IS accessible. 
The issue is that we can't access the protected member from THAT 
reference.

Easiest solution for now is probably to just make that method public.
08/20/2015 04:13:43 PM Michael Rubinsky Comment #5
State ⇒ Assigned
Assigned to Michael Slusarz
Reply to this comment
The INSTALL docs say it is HIGHLY RECOMMENDED to  use 5.4+. Being 
compatible with the API does not equal assurances that issues resolved 
in later versions will work correctly in Horde when still using 
earlier versions.

That being said, I was under the impression that it was *supposed* to 
work under 5.3.x and was a bug in PHP that it did not. However, I now 
see that the PHP docs state that it wasn't added until 5.4.

Assigning this to the maintainer of that code for further review.
08/20/2015 03:50:53 PM o+horde (at) immerda (dot) ch Comment #4 Reply to this comment
Sorry, I meant to say to upgrade PHP. This was recently discussed on
ok, well apparently this is only supported in >= 5.4 (see [0] and [1]).

one would expect that you announce that from now on 5.3 is not 
supported anymore and remove the sentence "Although the PHP 5.3 API is 
supported," from [2], or not?

since until right now we never saw a single incompatibility with 5.3....


[0] http://stackoverflow.com/questions/5734011/php-5-4-closure-this-support
[1] https://3v4l.org/neHqH
[2] http://www.horde.org/apps/horde/docs/INSTALL
08/20/2015 02:40:54 PM Michael Rubinsky Comment #3
Assigned to Michael Rubinsky
Reply to this comment
Sorry, I meant to say to upgrade PHP. This was recently discussed on 
the mailing list somewhere. This is probably due to you having a PHP 
version that didn't properly support the "$this" reference inside of 
closures.
08/20/2015 02:32:10 PM Michael Rubinsky Comment #2
Priority ⇒ 1. Low
State ⇒ Not A Bug
Reply to this comment
Please upgrade your install.
08/20/2015 11:40:32 AM o+horde (at) immerda (dot) ch Comment #1
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ gpg mail display broken
Queue ⇒ Horde Base
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
PHP Fatal error:  Call to protected method 
Horde_Crypt_Pgp_Backend_Binary::_callGpg() from context '' in 
pear/php/Horde/Crypt/Pgp/Backend/Binary.php on line 155

Saved Queries