OO-3029: don't close the panel if the user cancelled the closing of the entry,...

OO-3029: don't close the panel if the user cancelled the closing of the entry, enhance behavior of the multi filter with "show all"

src/main/java/org/olat/core/gui/components/form/flexible/impl/elements/table/FlexiTableElementImpl.java

@@ -1082,10 +1082,28 @@ public class FlexiTableElementImpl extends FormItemImpl implements FlexiTableEle
 				filter.setSelected(false);
 			}
 		} else if(multiFilterSelection) {
+			boolean showAll = false;
 			for(FlexiTableFilter filter:filters) {
-				if(filter.getFilter().equals(filterKey)) {
-					filter.setSelected(!filter.isSelected());
+				if(filter.getFilter().equals(filterKey) && filter.isShowAll()) {
+					showAll = !filter.isSelected();//Show all is currently not selected, but the event will toggle it
 				}
+			}
+			
+			if(showAll) {
+				for(FlexiTableFilter filter:filters) {
+					filter.setSelected(filter.isShowAll());
+				}
+			} else {
+				for(FlexiTableFilter filter:filters) {
+					if(filter.isShowAll()) {
+						filter.setSelected(false);
+					} else if(filter.getFilter().equals(filterKey)) {
+						filter.setSelected(!filter.isSelected());
+					}
+				}
+			}
+
+			for(FlexiTableFilter filter:filters) {
 				if(filter.isSelected()) {
 					selectedFilters.add(filter);
 				}

src/main/java/org/olat/modules/portfolio/ui/shared/SharedPagesController.java

@@ -311,9 +311,11 @@ public class SharedPagesController extends FormBasicController implements Activa
 	@Override
 	protected void event(UserRequest ureq, Controller source, Event event) {
 		if(pageCtrl == source) {
-			if(event instanceof ClosePageEvent || event instanceof DonePageEvent) {
+			if(event instanceof ClosePageEvent) {
 				loadModel(false, false);
 				stackPanel.popController(pageCtrl);
+			} else if(event instanceof DonePageEvent) {
+				loadModel(false, false);
 			}
 		}
 		super.event(ureq, source, event);

src/main/java/org/olat/modules/portfolio/ui/shared/SharedPagesDataModel.java

@@ -68,7 +68,9 @@ public class SharedPagesDataModel extends DefaultFlexiTableDataModel<SharedPageR
 	@Override
 	public void filter(List<FlexiTableFilter> filters) {
 		String key = filters == null || filters.isEmpty() || filters.get(0) == null ? null : filters.get(0).getFilter();
-		if(StringHelper.containsNonWhitespace(key)) {
+		if("all".equals(key)) {
+			super.setObjects(backups);
+		} else if(StringHelper.containsNonWhitespace(key)) {
 			List<String> allowedStatus = filters.stream()
 					.map(f -> f.getFilter()).collect(Collectors.toList());
 			List<SharedPageRow> filteredRows = backups.stream()