Implemented Publishing Datasets in finalised DMP's (Ticket #47 - Publishing Datasets - DMP Finalisation)

dmp-backend/data/src/main/java/eu/eudat/data/dao/entities/DatasetDaoImpl.java

@@ -63,8 +63,7 @@ public class DatasetDaoImpl extends DatabaseAccess<Dataset> implements DatasetDa
     public QueryableList<Dataset> getAuthenticated(QueryableList<Dataset> query, UserInfo principal) {
         if (principal.getId() == null) query.where((builder, root) -> builder.equal(root.get("isPublic"), true));
         else {
-            query.where((builder, root) -> builder.or(builder.equal(root.get("dmp").get("creator"), principal), builder.isMember(principal, root.get("dmp").get("users"))
+            query.where((builder, root) -> builder.or(builder.equal(root.get("dmp").get("creator"), principal), builder.isMember(principal, root.get("dmp").get("users"))));
-                    , builder.equal(root.get("isPublic"), true)));
         }
         return query;
     }

dmp-backend/data/src/main/java/eu/eudat/data/entities/Dataset.java

@@ -44,7 +44,7 @@ public class Dataset implements DataEntity<Dataset, UUID> {
     private static final Set<String> hints = new HashSet<>(Arrays.asList("datasetListingModel"));
 
     public enum Status {
-        SAVED((short) 0), FINALISED((short) 1), WITHDRAWN((short) 2), DELETED((short) 99);
+        SAVED((short) 0), FINALISED((short) 1), DELETED((short) 99);
 
         private short value;
 
@@ -62,8 +62,6 @@ public class Dataset implements DataEntity<Dataset, UUID> {
                     return SAVED;
                 case 1:
                     return FINALISED;
-                case 2:
-                    return WITHDRAWN;
                 case 99:
                     return DELETED;
                 default:

dmp-backend/data/src/main/java/eu/eudat/data/query/items/table/dataset/DatasetPublicTableRequest.java

@@ -1,6 +1,7 @@
 package eu.eudat.data.query.items.table.dataset;
 
 import eu.eudat.data.dao.criteria.DatasetPublicCriteria;
+import eu.eudat.data.entities.DMP;
 import eu.eudat.data.entities.Dataset;
 import eu.eudat.data.query.definition.TableQuery;
 import eu.eudat.queryable.QueryableList;
@@ -19,6 +20,8 @@ public class DatasetPublicTableRequest extends TableQuery<DatasetPublicCriteria,
     @Override
     public QueryableList<Dataset> applyCriteria() {
         QueryableList<Dataset> query = this.getQuery();
+        query.where((builder, root) -> builder.equal(root.get("dmp").get("status"), DMP.DMPStatus.FINALISED.getValue()));
+        query.where((builder, root) -> builder.equal(root.get("status"), Dataset.Status.FINALISED.getValue()));
         query.initSubQuery(String.class).where((builder, root) -> builder.equal(root.get("dmp").get("version"),
                 query.<String>subQueryMax((builder1, externalRoot, nestedRoot) -> builder1.equal(externalRoot.get("dmp").get("groupId"), nestedRoot.get("dmp").get("groupId")),
                         Arrays.asList(new SelectionField(FieldSelectionType.COMPOSITE_FIELD, "dmp:version")), String.class)));

dmp-backend/web/src/main/java/eu/eudat/logic/managers/DataManagementPlanManager.java

@@ -172,12 +172,6 @@ public class DataManagementPlanManager {
                     .asQueryable().where((builder, root) -> root.get("id").in(dataManagementPlan.getDatasets()
                     .stream().map(x -> UUID.fromString(x.getId())).collect(Collectors.toList())))
                     .update(root -> root.<Integer>get("status"), Dataset.Status.FINALISED.getValue());
-            apiContext.getOperationsContext().getDatabaseRepository().getDatasetDao()
-                    .asQueryable().where((builder, root) -> builder.and(
-                        builder.equal(root.get("dmp").get("id"), dataManagementPlan.getId()),
-                        builder.not(root.get("id").in(dataManagementPlan.getDatasets().stream().map(x -> UUID.fromString(x.getId())).collect(Collectors.toList()))),
-                        builder.equal(root.get("status"), Dataset.Status.SAVED.getValue())))
-                    .update(root -> root.<Integer>get("status"), Dataset.Status.WITHDRAWN.getValue());
         }
         if (dataManagementPlan.getAssociatedUsers().stream().filter(item -> item.getId() == principal.getId()).collect(Collectors.toList()).size() == 0)
             assignUser(newDmp, user, apiContext);

dmp-backend/web/src/main/java/eu/eudat/logic/managers/DatasetManager.java

@@ -344,7 +344,6 @@ public class DatasetManager {
         eu.eudat.data.entities.Dataset dataset = datasetDao.find(id);
         if (dataset.getStatus() != eu.eudat.data.entities.Dataset.Status.FINALISED.getValue())
             throw new Exception("You cannot make public a Dataset That Has not Been Finalised");
-        dataset.setPublic(true);
         datasetDao.createOrUpdate(dataset);
     }
 

dmp-frontend/src/app/datasets/dataset-public/dataset-public-listing.component.html

@@ -35,12 +35,6 @@
 							<mat-cell *matCellDef="let row"> {{row.profile}} </mat-cell>
 						</ng-container>
 
-						<!-- Column Definition: Status -->
-						<ng-container cdkColumnDef="status">
-							<mat-header-cell *matHeaderCellDef>{{'DATASET-LISTING.COLUMNS.STATUS' | translate}}</mat-header-cell>
-							<mat-cell *matCellDef="let row"> {{row.status}} </mat-cell>
-						</ng-container>
-
 						<!-- Column Definition: DataRepositories -->
 						<!-- <ng-container cdkColumnDef="dataRepositories">
               <mat-header-cell *matHeaderCellDef>{{'DATASET-LISTING.COLUMNS.DATAREPOSITORIES' | translate}}</mat-header-cell>

dmp-frontend/src/app/datasets/dataset-public/dataset-public-listing.component.ts

@@ -27,7 +27,7 @@ export class DatasetPublicListingComponent implements OnInit {
 	criteria: FacetSearchCriteriaModel;
 
 	dataSource: DatasetDataSource | null;
-	displayedColumns: String[] = ['label', 'project', 'profile', 'status', 'description', 'created'];
+	displayedColumns: String[] = ['label', 'project', 'profile', 'description', 'created'];
 	pageEvent: PageEvent;
 	titlePrefix: String;
 	dmpId: string;

dmp-frontend/src/app/datasets/listing/dataset-listing.component.html

@@ -79,25 +79,6 @@
 				<mat-cell *matCellDef="let row">{{row.created | date:'shortDate'}}</mat-cell>
 			</ng-container>
 
-      <!-- Column Definition: Submission Time -->
-      <ng-container cdkColumnDef="actions">
-        <mat-header-cell *matHeaderCellDef>{{'DATASET-LISTING.COLUMNS.ACTIONS' | translate}}</mat-header-cell>
-        <mat-cell *matCellDef="let row" (click)="$event.stopPropagation()">
-          <mat-menu #actionsMenu="matMenu">
-            <button *ngIf="row.status == 0" (click)="rowClick(row.id)" mat-menu-item>
-              <mat-icon>mode_edit</mat-icon>{{'DATASET-LISTING.ACTIONS.EDIT' | translate}}
-            </button>
-            <button *ngIf="row.status != 0" (click)="makeItPublic(row.id)" mat-menu-item>
-              <mat-icon>public</mat-icon>{{'DATASET-LISTING.ACTIONS.MAKE-IT-PUBLIC' | translate}}
-            </button>
-
-          </mat-menu>
-          <button mat-icon-button [matMenuTriggerFor]="actionsMenu">
-            <mat-icon>more_vert</mat-icon>
-          </button>
-        </mat-cell>
-      </ng-container>
-
 			<mat-header-row *matHeaderRowDef="displayedColumns"></mat-header-row>
 			<mat-row *matRowDef="let row; columns: displayedColumns" (click)="rowClick(row.id)"></mat-row>
 			<!-- (click)="rowClick(row.id)" -->

dmp-frontend/src/app/datasets/listing/dataset-listing.component.ts

@@ -35,7 +35,7 @@ export class DatasetListingComponent implements OnInit, IBreadCrumbComponent {
 	breadCrumbs: Observable<BreadcrumbItem[]>;
 
 	dataSource: DatasetDataSource | null;
-	displayedColumns: String[] = ['label', 'project', 'dmp', 'profile', 'status', 'description', 'created', 'actions'];
+	displayedColumns: String[] = ['label', 'project', 'dmp', 'profile', 'status', 'description', 'created'];
 	pageEvent: PageEvent;
 	titlePrefix: String;
 	dmpId: string;

dmp-frontend/src/app/models/datasets/DatasetWizardModel.ts

@@ -17,7 +17,6 @@ import { TagModel } from '../tags/TagModel';
 export enum DatasetStatus {
 	Draft = 0,
 	Finalised = 1,
-	WithDrawn = 2,
 	Deleted = 99
 }
 

dmp-frontend/src/app/utilities/utilities.ts

@@ -39,7 +39,6 @@ export class Utilities {
 		switch (status) {
 			case DatasetStatus.Draft: return this.language.instant('TYPES.DATASET-STATUS.DRAFT');
 			case DatasetStatus.Finalised: return this.language.instant('TYPES.DATASET-STATUS.FINALISED');
-			case DatasetStatus.WithDrawn: return this.language.instant('TYPES.DATASET-STATUS.WITHDRAWN');
 		}
 	}
 

dmp-frontend/src/assets/lang/en.json

@@ -285,8 +285,7 @@
 		},
 		"DATASET-STATUS": {
 			"DRAFT": "Draft",
-			"FINALISED": "Finalised",
+			"FINALISED": "Finalised"
-			"WITHDRAWN": "Withdrawn"
 		},
 		"EXTERNAL-DATASET-TYPE": {
 			"SOURCE": "Source",