[directory-fortress-commander] branch FC-281 updated: fix optional fallback link

[sm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

smckinney pushed a commit to branch FC-281
in repository https://gitbox.apache.org/repos/asf/directory-fortress-commander.git


The following commit(s) were added to refs/heads/FC-281 by this push:
     new dcbc810  fix optional fallback link
dcbc810 is described below

commit dcbc810bd82b7239d2946164fd937e6f284e2706
Author: Shawn McKinney <sm...@symas.com>
AuthorDate: Wed May 13 11:17:40 2020 -0500

    fix optional fallback link
---
 .../org/apache/directory/fortress/web/panel/GroupDetailPanel.java    | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/apache/directory/fortress/web/panel/GroupDetailPanel.java b/src/main/java/org/apache/directory/fortress/web/panel/GroupDetailPanel.java
index b10e112..3efad36 100644
--- a/src/main/java/org/apache/directory/fortress/web/panel/GroupDetailPanel.java
+++ b/src/main/java/org/apache/directory/fortress/web/panel/GroupDetailPanel.java
@@ -163,7 +163,7 @@ public class GroupDetailPanel extends FormComponentPanel
                     AjaxFallbackLink<Member> removeLink = new AjaxFallbackLink<Member>( "remove-member", item.getModel() )
                     {
                         @Override
-                        public void onClick(Optional<AjaxRequestTarget> target)
+                        public void onClick(Optional<AjaxRequestTarget> targetOptional)
                         {
                             Member member = item.getModelObject();
                             Group group = ( Group ) editForm.getModel().getObject();
@@ -184,8 +184,7 @@ public class GroupDetailPanel extends FormComponentPanel
                                 display.setMessage( szError );
                                 log.warn( szError );
                             }
-                            // TODO: fix me, figure out how to add a component with optional target:
-                            //target.add( component );
+                            targetOptional.ifPresent(target -> target.add( component ));
                         }
                     };
                     removeLink.setOutputMarkupId( true );