Merge pull request #290 from dinurp/main

chgeo · web-flow · commit ce752097d044 · 2021-11-10T08:55:47.000+01:00
Annotations for Admin.Book service
diff --git a/bookshop/srv/admin-service.cds b/bookshop/srv/admin-service.cds
@@ -3,3 +3,7 @@ service AdminService @(requires:'admin') {
   entity Books as projection on my.Books;
   entity Authors as projection on my.Authors;
 }
+
+//Since ID is computed, we can hide the popup for ID on Create
+annotate AdminService.Books with   { ID @Core.Computed; }
+annotate AdminService.Authors with { ID @Core.Computed; }
diff --git a/fiori/app/admin/fiori-service.cds b/fiori/app/admin/fiori-service.cds
@@ -85,7 +85,8 @@ annotate AdminService.Books.texts with @(
 
 // Add Value Help for Locales
 annotate AdminService.Books.texts {
-	locale @ValueList:{entity:'Languages',type:#fixed}
+	locale @ValueList:{entity:'Languages'};
+	locale @Common.ValueListWithFixedValues:true; //show as drop down, not a dialog
 }
 // In addition we need to expose Languages through AdminService as a target for ValueList
 using { sap } from '@sap/cds/common';

Original file line number	Diff line number	Diff line change
`@@ -3,3 +3,7 @@ service AdminService @(requires:'admin') {`
`3`	`3`	`entity Books as projection on my.Books;`
`4`	`4`	`entity Authors as projection on my.Authors;`
`5`	`5`	`}`
	`6`	`+`
	`7`	`+//Since ID is computed, we can hide the popup for ID on Create`
	`8`	`+annotate AdminService.Books with { ID @Core.Computed; }`
	`9`	`+annotate AdminService.Authors with { ID @Core.Computed; }`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,8 @@ annotate AdminService.Books.texts with @(`
`85`	`85`
`86`	`86`	`// Add Value Help for Locales`
`87`	`87`	`annotate AdminService.Books.texts {`
`88`		`- locale @ValueList:{entity:'Languages',type:#fixed}`
	`88`	`+ locale @ValueList:{entity:'Languages'};`
	`89`	`+ locale @Common.ValueListWithFixedValues:true; //show as drop down, not a dialog`
`89`	`90`	`}`
`90`	`91`	`// In addition we need to expose Languages through AdminService as a target for ValueList`
`91`	`92`	`using { sap } from '@sap/cds/common';`