Error when we configure view to contain only 1 column

Description

Error if the configuration view contain only 1 column

Distribution Name and Version

Debian stretch

FusionDirectory Version

1.4

PHP version used

7

Origin of php packages

Debian

Steps to Reproduce

1. Remove all the column
2. Save the configuration and get the error

Expected behavior:

No error

Actual behavior:

We get this error

 Fatal error: Uncaught Error: Call to a member function setTemplateAttributes() on null in /usr/share/fusiondirectory/include/management/class_managementListing.inc:114 
Stack trace: 
#0 /usr/share/fusiondirectory/include/management/class_management.inc(361): managementListing->reloadColumns() 
#1 /usr/share/fusiondirectory/include/management/class_ManagementConfigurationDialog.inc(223): management->setColumnConfiguration(Array) 
#2 /usr/share/fusiondirectory/include/management/class_ManagementDialog.inc(52): ManagementConfigurationDialog->save() 
#3 /usr/share/fusiondirectory/include/management/class_ManagementDialog.inc(41): ManagementDialog->handle_finish() 
#4 /usr/share/fusiondirectory/include/management/class_management.inc(455): ManagementDialog->save_object() 
#5 /usr/share/fusiondirectory/include/management/class_management.inc(1266): management->execute() 
#6 /usr/share/fusiondirectory/include/class_pluglist.inc(564): management::mainInc() 
#7 /usr/share/fusiondirectory/html/main.php(225): pluglist::runMainInc('80') 
#8 {m in /usr/share/fusiondirectory/include/management/class_managementListing.inc on line 114

Reproduces how often: 100%

changed milestone to %FusionDirectory 1.4

By Jonathan Swaelens on 2019-06-17T12:42:46 (imported from GitLab)

added PJ1802-0188 label

changed the description

By bmortier on 2019-06-20T15:53:16 (imported from GitLab)

changed due date to June 24, 2019

By bmortier on 2019-06-20T15:53:21 (imported from GitLab)

changed title from Error if th configuration view contain only 1 column to Error when we configure view to contain only 1 column

By Côme Chilliet on 2019-07-15T12:42:50 (imported from GitLab)

changed the description

By Côme Chilliet on 2019-07-15T12:42:50 (imported from GitLab)

created merge request !630 to address this issue

By Côme Chilliet on 2019-07-15T12:43:09 (imported from GitLab)

mentioned in merge request !630

By Côme Chilliet on 2019-07-15T12:43:09 (imported from GitLab)

mentioned in commit 29f8a9ed

By Côme Chilliet on 2019-07-15T12:58:38 (imported from GitLab)

added 30m of time spent at 2019-07-15

By Côme Chilliet on 2019-07-15T15:39:01 (imported from GitLab)

added To Be Tested label

assigned to @jswaelens and unassigned @MCMic

By Côme Chilliet on 2019-07-16T12:18:06 (imported from GitLab)

Hello,

If I remove all the collumn I have this error

<br />
<b>Fatal error</b>:  Uncaught TypeError: Argument 2 passed to EntrySortIterator::__construct() must be an instance of Column, null given, called in /usr/share/fusiondirectory/include/management/class_managementListing.inc on line 230 and defined in /usr/share/fusiondirectory/include/management/class_EntrySortIterator.inc:39
Stack trace:
#0 /usr/share/fusiondirectory/include/management/class_managementListing.inc(230): EntrySortIterator-&gt;__construct(Array, NULL, false)
#1 /usr/share/fusiondirectory/include/management/class_managementListing.inc(179): managementListing-&gt;getIterator()
#2 /usr/share/fusiondirectory/include/management/class_management.inc(494): managementListing-&gt;render()
#3 /usr/share/fusiondirectory/include/management/class_management.inc(486): management-&gt;renderList()
#4 /usr/share/fusiondirectory/include/management/class_management.inc(1271): management-&gt;execute()
#5 /usr/share/fusiondirectory/include/class_pluglist.inc(561): management::mainInc()
#6 /usr/share/fusiondirectory/html/main.php(223): pluglist::runMain in <b>/usr/share/fusiondirectory/include/management/class_EntrySortIterator.inc</b> on line <b>39</b><br />

By Jonathan Swaelens on 2019-07-16T14:06:44 (imported from GitLab)

added 10m of time spent at 2019-07-16

By Jonathan Swaelens on 2019-07-16T14:06:45 (imported from GitLab)

removed To Be Tested label

assigned to @MCMic and unassigned @jswaelens

By Jonathan Swaelens on 2019-07-16T14:06:45 (imported from GitLab)

created merge request !632 to address this issue

By Côme Chilliet on 2019-07-16T14:47:06 (imported from GitLab)

mentioned in merge request !632

By Côme Chilliet on 2019-07-16T14:47:06 (imported from GitLab)

mentioned in commit 0ed1eefd

By Côme Chilliet on 2019-07-16T14:53:30 (imported from GitLab)

Should be good

By Côme Chilliet on 2019-07-16T15:22:23 (imported from GitLab)