GPG key addition crash

Description

Uploading of a PGP Key via the web interface caused an exception.

Distribution Name and Version

Ubuntu 16.04

FusionDirectory Version

1.3

Plugin with the defect

gpg

PHP version used

7.0.4

Origin of php packages

Debian Repository debian-stretch from repos.fusiondirectory.org

Steps to Reproduce

1. Login to Fusion Directory
2. Click on GPG
3. Click on Add button
4. Select an entry and add it

Expected behavior:

Key is added

Actual behavior:

Exception is thrown.

Reproduces how often: 100%

Additional Information

I've provided a fix: pthomas/fd-plugins@3618a5a0

Unfortunately I'm not able to make a merge request. The fix is tested in our environment.

assigned to @bmortier

assigned to @MCMic and unassigned @bmortier

changed due date to July 22, 2019

added PJ1802-0188 plugin-gpg labels

Stack trace on 1.4:

<b>Fatal error</b>:  Uncaught TypeError: Argument 1 passed to pgpKeySelect::filterSingleValue() must be of the type array, string given, called in /usr/share/fusiondirectory/plugins/personal/gpg/pgpKeySelect/class_pgpKeySelect.inc on line 86 and defined in /usr/share/fusiondirectory/plugins/personal/gpg/pgpKeySelect/class_pgpKeySelect.inc:75
Stack trace:
#0 /usr/share/fusiondirectory/plugins/personal/gpg/pgpKeySelect/class_pgpKeySelect.inc(86): pgpKeySelect::filterSingleValue('', 'pgpDate')
#1 /usr/share/fusiondirectory/include/class_listing.inc(827): pgpKeySelect::filterPgpDate('')
#2 /usr/share/fusiondirectory/include/class_listing.inc(738): listing-&gt;processElementFilter('column', 3, '%{filter:pgpDat...', Array, 0)
#3 /usr/share/fusiondirectory/include/class_listing.inc(406): listing-&gt;renderCell('column', 3, '%{filter:pgpDat...', Array, 0)
#4 /usr/share/fusiondirectory/include/simpleplugin/class_simpleManagement.inc(538): listing-&gt;render()
#5 /usr/share/fusiondirectory/include/simpleplugin/class_simpleManagement.inc(595): simpl in <b>/usr/share/fusiondirectory/plugins/personal/gpg/pgpKeySelect/class_pgpKeySelect.inc</b> on line <b>75</b><br />

changed milestone to %FusionDirectory 1.3.2

mentioned in commit 365ee0a5

mentioned in commit 7f4684d5

added 1h 30m of time spent at 2019-07-23

added To Be Tested + 1 deleted label

assigned to @jswaelens and unassigned @MCMic

changed milestone to %FusionDirectory 1.3.1

Tested on demo-fixes and no more crash

closed

added 45m of time spent at 2019-08-29

removed due date

added Fixed label and removed To Be Tested label

added Authors label