@sap/cds-compiler 2.15.2 → 3.0.2

package/lib/model/revealInternalProperties.js

@@ -14,6 +14,7 @@
 const msg = require('../base/messages');
 
 const $inferred = Symbol.for('cds.$inferred');
+const $location = Symbol.for('cds.$location');
 
 class NOT_A_DICTIONARY {}       // used for consol.log display
 
@@ -96,6 +97,7 @@ function revealInternalProperties( model, nameOrPath ) {
     _annotate: reveal,
     _deps: dependencyInfo,
     _status: primOrString,       // is a string anyway
+    $annotations: as => as.map( $annotation ),
     $messageFunctions: () => '‹some functions›',
   }
   unique_id = 1;
@@ -129,7 +131,10 @@ function revealInternalProperties( model, nameOrPath ) {
 
     path = path.split('/');
     if (path.length === 1) {
-      return reveal( xsn.definitions[path] || xsn.vocabularies && xsn.vocabularies[path] );
+      const def = xsn.definitions?.[path[0]] || xsn.vocabularies?.[path[0]];
+      if (!def)
+        throw new Error(`reveal xsn: Unknown definition: “${path[0]}”`)
+      return reveal( def );
     }
 
     // with the code below, we might miss the right transformer function
@@ -149,7 +154,7 @@ function revealInternalProperties( model, nameOrPath ) {
 
   function shortenName( node, parent ) {
     const name = reveal( node, parent );
-    if (name && typeof name === 'object' && name.absolute) {
+    if (name && typeof name === 'object' && name.absolute && node.kind) {
       const text = artifactIdentifier( parent );
       delete name.absolute;
       delete name.select;
@@ -178,11 +183,20 @@ function revealInternalProperties( model, nameOrPath ) {
     return r;
   }
 
+  function $annotation( anno ) { // property for cds-lsp
+    const { name, $flatten } = anno.value || anno;
+    const value = ($flatten)
+          ? { name: reveal( name ), $flatten: $flatten.map( $annotation ) }
+          : `@${name?.absolute}`;
+    return { value, location: locationString( anno.location || anno.name.location ) };
+  }
+
   function columns( nodes, query ) {
     // If we will have specified elements, we need another test to see columns in --parse-cdl
-    return nodes && nodes.map( c => (c._parent && c._parent.elements)
-                                    ? artifactIdentifier( c, query )
-                                    : reveal( c, nodes ) );
+    return nodes && array( nodes,
+                           c => (c._parent && c._parent.elements)
+                           ? artifactIdentifier( c, query )
+                           : reveal( c, nodes ) );
   }
 
   function elements( dict, parent ) {
@@ -215,7 +229,7 @@ function revealInternalProperties( model, nameOrPath ) {
     if (!node || typeof node !== 'object')
       return primOrString( node );
     if (Array.isArray(node))  // with args
-      return node.map( n => reveal( n, node ) );
+      return array( node, reveal );
     // Make unexpected prototype visible with node-10+:
     const r = Object.create( Object.getPrototypeOf(node)
                              ? NOT_A_DICTIONARY.prototype
@@ -225,6 +239,8 @@ function revealInternalProperties( model, nameOrPath ) {
     }
     if (node[$inferred] && !node['[$inferred]'])
       r['[$inferred]'] = node[$inferred];
+    if (node[$location] && !node['[$location]'])
+      r['[$location]'] = locationString( node[$location] );
     return r;
   }
 
@@ -239,7 +255,7 @@ function revealInternalProperties( model, nameOrPath ) {
     if (node == null || typeof node !== 'object' )
       return primOrString( node );
     if (Array.isArray(node))
-      return node.map( n => revealNonEnum( n, node ) );
+      return array( node, revealNonEnum );
 
     if (Object.getPrototypeOf( node ))
       return artifactIdentifier( node, parent );
@@ -250,7 +266,7 @@ function revealInternalProperties( model, nameOrPath ) {
     if (node == null || typeof node !== 'object' )
       return node
     if (Array.isArray(node))
-      return node.map( n => reveal( n, node, name ) );
+      return array( node, n => reveal( n, node, name ) );
 
     const asLinkTest = kindsRepresentedAsLinks[ node.kind ];
     if (asLinkTest && asLinkTest( node, parent, name ))
@@ -280,6 +296,13 @@ function revealInternalProperties( model, nameOrPath ) {
   }
 }
 
+function array( node, fn ) {
+  const r = node.map( n => fn( n, node ) );
+  if (node[$location])
+    r.push( { $location: locationString( node[$location] ) } );
+  return r;
+}
+
 function artifactIdentifier( node, parent ) {
   if (Array.isArray(node))
     return node.map( a => artifactIdentifier( a, node ) );
@@ -335,7 +358,7 @@ function primOrString( node ) {
   if (node == null || typeof node !== 'object')
     return node
   if (Array.isArray(node))
-    return node.map( primOrString );
+    return array( node, primOrString );
   if (Object.getPrototypeOf( node ))
     return '' + node;
   else

package/lib/model/sortViews.js

@@ -21,7 +21,7 @@ const { ModelError } = require("../base/error");
  */
 function sortTopologically(csn, _dependents, _dependencies){
   const layers = [];
-  let { zero, nonZero } = calculateDepth(Object.entries(csn.definitions));
+  let { zero, nonZero } = _calculateDepth(Object.entries(csn.definitions), _dependents, _dependencies);
   while (zero.length !== 0){
     const currentLayer = [];
     zero.forEach(([artifactName, artifact]) => {
@@ -34,46 +34,46 @@ function sortTopologically(csn, _dependents, _dependencies){
       }
     });
     layers.push(currentLayer);
-    ({zero, nonZero} = findWithXPointers(nonZero, 0));
+    ({zero, nonZero} = _findWithXPointers(nonZero, 0, _dependents, _dependencies));
   }
 
   return { layers, leftover: nonZero };
+}
 
-  function calculateDepth(definitionsArray) {
-    const zero = [];
-    const nonZero = [];
+function _calculateDepth(definitionsArray, _dependents, _dependencies) {
+  const zero = [];
+  const nonZero = [];
 
-    definitionsArray.forEach(([artifactName, artifact]) => {
-      if(artifact[_dependencies]) {
-        artifact.$pointers = artifact[_dependencies].size;
-        nonZero.push([artifactName, artifact]);
-      } else {
-        artifact.$pointers = 0;
-        zero.push([artifactName, artifact]);
-      }
-    });
-    return {
-      zero,
-      nonZero
+  definitionsArray.forEach(([artifactName, artifact]) => {
+    if(artifact[_dependencies]) {
+      artifact.$pointers = artifact[_dependencies].size;
+      nonZero.push([artifactName, artifact]);
+    } else {
+      artifact.$pointers = 0;
+      zero.push([artifactName, artifact]);
     }
+  });
+  return {
+    zero,
+    nonZero
   }
+}
 
-  function findWithXPointers(definitionsArray, x=0){
-    const zero = [];
-    const nonZero = [];
-
-    definitionsArray.forEach(([artifactName, artifact]) => {
-      if(artifact.$pointers !== undefined && artifact.$pointers === x) {
-        zero.push([artifactName, artifact]);
-      } else {
-        nonZero.push([artifactName, artifact]);
-      }
-    });
+function _findWithXPointers(definitionsArray, x, _dependents, _dependencies){
+  const zero = [];
+  const nonZero = [];
 
-    return {
-      zero,
-      nonZero
+  definitionsArray.forEach(([artifactName, artifact]) => {
+    if(artifact.$pointers !== undefined && artifact.$pointers === x) {
+      zero.push([artifactName, artifact]);
+    } else {
+      nonZero.push([artifactName, artifact]);
     }
+  });
+
+  return {
+    zero,
+    nonZero
   }
 }
 
@@ -83,6 +83,7 @@ function sortTopologically(csn, _dependents, _dependencies){
  * be run beforehand to resolve association usages.
  *
  * @param {object} sql Map of <object name>: "CREATE STATEMENT"
+ * @param {CSN.Model} csn
  *
  * @returns {{name: string, sql: string}[]} Sorted array of artifact name / "CREATE STATEMENTS" pairs
  *

package/lib/optionProcessor.js

@@ -19,10 +19,7 @@ optionProcessor
   .option('    --color <mode>', ['auto', 'always', 'never'])
   .option('-o, --out <dir>')
   .option('    --cds-home <dir>')
-  .option('    --lint-mode')
   .option('    --fuzzy-csn-error')
-  .option('    --trace-parser')
-  .option('    --trace-parser-amb')
   .option('    --trace-fs')
   .option('    --error <id-list>')
   .option('    --warn <id-list>')
@@ -34,9 +31,7 @@ optionProcessor
   .option('    --beta-mode')
   .option('    --beta <list>')
   .option('    --deprecated <list>')
-  .option('    --hana-flavor')
   .option('    --direct-backend')
-  .option('    --parse-only')
   .option('    --fallback-parser <type>', ['cdl', 'csn', 'csn!'])
   .option('    --test-mode')
   .option('    --test-sort-csn')
@@ -79,7 +74,6 @@ optionProcessor
                               never:
    -o, --out <dir>          Place generated files in directory <dir>, default is "-" for <stdout>
        --cds-home <dir>     When set, modules starting with '@sap/cds/' are searched in <dir>
-       --lint-mode          Generate nothing, just produce messages if any (for use by editors)
        --fuzzy-csn-error    Report free-style CSN properties as errors
        --                   Indicate the end of options (helpful if source names start with "-")
 
@@ -88,8 +82,6 @@ optionProcessor
        --default-string-length <length> Default 'length' for 'cds.String'
 
   Diagnostic options
-       --trace-parser       Trace parser
-       --trace-parser-amb   Trace parser ambiguities
        --trace-fs           Trace file system access caused by "using from"
 
   Severity options
@@ -107,22 +99,12 @@ optionProcessor
        --beta-mode          Enable all unsupported, incomplete (beta) features
        --beta <list>        Comma separated list of unsupported, incomplete (beta) features to use.
                             Valid values are:
-                              addTextsLanguageAssoc
                               hanaAssocRealCardinality
                               mapAssocToJoinCardinality
                               ignoreAssocPublishingInUnion
        --deprecated <list>  Comma separated list of deprecated options.
                             Valid values are:
-                              noElementsExpansion
-                              v1KeysForTemporal
-                              parensAsStrings
-                              projectionAsQuery
-                              renderVirtualElements
-                              unmanagedUpInComponent
-                              createLocalizedViews
-                              redirectInSubQueries
-       --hana-flavor        Compile with backward compatibility for HANA CDS (incomplete)
-       --parse-only         Stop compilation after parsing and write result to <stdout>
+                              eagerPersistenceForGeneratedEntities
        --fallback-parser <type>  If the language cannot be deduced by the file's extensions, use this
                                  parser as a fallback. Valid values are:
                                    cdl  : Use CDL parser
@@ -151,6 +133,8 @@ optionProcessor
        toCsn [options] <files...>               (default) Generate original model as CSN
        parseCdl [options] <file>                Generate a CSN that is close to the CDL source.
        explain <message-id>                     Explain a compiler message.
+       parseOnly [options] <files...>           (internal)  Stop compilation after parsing, write messages to <stderr>,
+                                                per default no output.
        toRename [options] <files...>            (internal) Generate SQL DDL rename statements
        manageConstraints [options] <files...>   (internal) Generate ALTER TABLE statements to
                                                            add / modify referential constraints.
@@ -159,7 +143,7 @@ optionProcessor
 // ----------- toHana -----------
 optionProcessor.command('H, toHana')
   .option('-h, --help')
-  .option('-n, --names <style>', ['plain', 'quoted', 'hdbcds'])
+  .option('-n, --sql-mapping <style>', ['plain', 'quoted', 'hdbcds'], { aliases: ['--names'] })
   .option('    --render-virtual')
   .option('    --joinfk')
   .option('-u, --user <user>')
@@ -176,7 +160,7 @@ optionProcessor.command('H, toHana')
 
   Options
    -h, --help                 Show this help text
-   -n, --names <style>        Naming style for generated entity and element names:
+   -n, --sql-mapping <style>  Naming style for generated entity and element names:
                                 plain  : (default) Produce HANA entity and element names in
                                          uppercase and flattened with underscores. Do not generate
                                          structured types.
@@ -207,7 +191,7 @@ optionProcessor.command('H, toHana')
 
 optionProcessor.command('O, toOdata')
   .option('-h, --help')
-  .option('-v, --version <version>', ['v2', 'v4', 'v4x'])
+  .option('-v, --odata-version <version>', ['v2', 'v4', 'v4x'], { aliases: ['--version'] })
   .option('-x, --xml')
   .option('-j, --json')
   .option('    --odata-containment')
@@ -217,7 +201,7 @@ optionProcessor.command('O, toOdata')
   .option('    --odata-v2-partial-constr')
   .option('-c, --csn')
   .option('-f, --odata-format <format>', ['flat', 'structured'])
-  .option('-n, --names <style>', ['plain', 'quoted', 'hdbcds'])
+  .option('-n, --sql-mapping <style>', ['plain', 'quoted', 'hdbcds'], { aliases: [ '--names' ] })
   .option('-s, --service-names <list>')
   .help(`
   Usage: cdsc toOdata [options] <files...>
@@ -226,10 +210,10 @@ optionProcessor.command('O, toOdata')
 
   Options
    -h, --help               Show this help text
-   -v, --version <version>  ODATA version
-                              v2: ODATA V2
-                              v4: (default) ODATA V4
-                              v4x: { version: 'v4', odataContainment:true, format:'structured' }
+   -v, --odata-version <version>  ODATA version
+                                    v2: ODATA V2
+                                    v4: (default) ODATA V4
+                                    v4x: { version: 'v4', odataContainment:true, format:'structured' }
    -x, --xml                (default) Generate XML output (separate or combined)
    -j, --json               Generate JSON output as "<svc>.json" (not available for v2)
    -c, --csn                Generate "odata_csn.json" with ODATA-preprocessed model
@@ -242,15 +226,15 @@ optionProcessor.command('O, toOdata')
        --odata-foreign-keys        Render foreign keys in structured format (V4 only)
        --odata-v2-partial-constr   Render referential constraints also for partial principal key tuple
                                    (Not spec compliant and V2 only)
-   -n, --names <style>      Annotate artifacts and elements with "@cds.persistence.name", which is
-                            the corresponding database name (see "--names" for "toHana or "toSql")
-                              plain   : (default) Names in uppercase and flattened with underscores
-                              quoted  : Names in original case as in CDL. Entity names with dots,
-                                        but element names flattened with underscores
-                              hdbcds  : Names as HANA CDS would generate them from the same CDS
-                                        source (like "quoted", but using element names with dots)
+   -n, --sql-mapping <style> Annotate artifacts and elements with "@cds.persistence.name", which is
+                             the corresponding database name (see "--sql-mapping" for "toHana or "toSql")
+                               plain   : (default) Names in uppercase and flattened with underscores
+                               quoted  : Names in original case as in CDL. Entity names with dots,
+                                         but element names flattened with underscores
+                               hdbcds  : Names as HANA CDS would generate them from the same CDS
+                                         source (like "quoted", but using element names with dots)
     -s, --service-names <list>    List of comma-separated service names to be rendered
-                                  (default) empty, all services are rendered 
+                                  (default) empty, all services are rendered
 `);
 
 optionProcessor.command('C, toCdl')
@@ -266,10 +250,10 @@ optionProcessor.command('C, toCdl')
 
 optionProcessor.command('Q, toSql')
   .option('-h, --help')
-  .option('-n, --names <style>', ['plain', 'quoted', 'hdbcds'])
+  .option('-n, --sql-mapping <style>', ['plain', 'quoted', 'hdbcds'], { aliases: [ '--names' ] })
+  .option('-d, --sql-dialect <dialect>',  ['hana', 'sqlite', 'plain', 'postgres'], { aliases: [ '--dialect' ] })
   .option('    --render-virtual')
   .option('    --joinfk')
-  .option('-d, --dialect <dialect>', ['hana', 'sqlite', 'plain'])
   .option('-u, --user <user>')
   .option('-l, --locale <locale>')
   .option('-s, --src <style>', ['sql', 'hdi'])
@@ -286,7 +270,7 @@ optionProcessor.command('Q, toSql')
 
   Options
    -h, --help                 Show this help text
-   -n, --names <style>        Naming style for generated entity and element names:
+   -n, --sql-mapping <style>  Naming style for generated entity and element names:
                                 plain  : (default) Produce SQL table and view names in
                                          flattened with underscores format (no quotes required)
                                 quoted : Produce SQL table and view names in original case as in
@@ -299,10 +283,11 @@ optionProcessor.command('Q, toSql')
                                          combination with "hana" dialect.
        --render-virtual       Render virtual elements in views and draft tables
        --joinfk               Create JOINs for foreign key accesses
-   -d, --dialect <dialect>    SQL dialect to be generated:
-                                plain  : (default) Common SQL - no assumptions about DB restrictions
-                                hana   : SQL with HANA specific language features
-                                sqlite : Common SQL for sqlite
+   -d, --sql-dialect <dialect>  SQL dialect to be generated:
+                                plain    : (default) Common SQL - no assumptions about DB restrictions
+                                hana     : SQL with HANA specific language features
+                                sqlite   : Common SQL for sqlite
+                                postgres : Common SQL for postgres - beta-feature
    -u, --user <user>          Value for the "$user" variable
    -l, --locale <locale>      Value for the "$user.locale" variable in "sqlite"/"plain" dialect
    -s, --src <style>          Generate SQL source files as <artifact>.<suffix>
@@ -329,28 +314,29 @@ optionProcessor.command('Q, toSql')
 
 optionProcessor.command('toRename')
   .option('-h, --help')
-  .option('-n, --names <style>', ['quoted', 'hdbcds'])
+  .option('-n, --sql-mapping <style>', ['quoted', 'hdbcds'], { aliases: ['--names'] })
   .help(`
   Usage: cdsc toRename [options] <files...>
 
   (internal, subject to change): Generate SQL stored procedure containing DDL statements to
   "storedProcedure.sql" that allows to rename existing tables and their columns so that they
-  match the result of "toHana" or "toSql" with the "--names plain" option.
+  match the result of "toHana" or "toSql" with the "--sql-mapping plain" option.
 
   Options
    -h, --help           Display this help text
-   -n, --names <style>  Assume existing tables were generated with "--names <style>":
+   -n, --sql-mapping <style>
+                        Assume existing tables were generated with "--sql-mapping <style>":
                           quoted   : Assume existing SQL tables and views were named in original
                                      case as in CDL (with dots), but column names were flattened
-                                     with underscores (e.g. resulting from "toHana --names quoted")
+                                     with underscores (e.g. resulting from "toHana --sql-mapping quoted")
                           hdbcds   : (default) Assume existing SQL tables, views and columns were
                                      generated by HANA CDS from the same CDS source (or resulting
-                                     from "toHana --names hdbcds")
+                                     from "toHana --sql-mapping hdbcds")
 `);
 
 optionProcessor.command('manageConstraints')
   .option('-h, --help')
-  .option('-n, --names <style>', ['plain', 'quoted', 'hdbcds'])
+  .option('-n, --sql-mapping <style>', ['plain', 'quoted', 'hdbcds'], { aliases: ['--names'] })
   .option('-s, --src <style>', ['sql', 'hdi'])
   .option('    --drop')
   .option('    --alter')
@@ -366,7 +352,8 @@ optionProcessor.command('manageConstraints')
 
   Options
    -h, --help             Display this help text
-   -n, --names <style>    Assume existing tables were generated with "--names <style>":
+   -n, --sql-mapping <style>
+                          Assume existing tables were generated with "--sql-mapping <style>":
                             plain    : (default) Assume SQL tables were flattened and dots were
                                         replaced by underscores
                             quoted   : Assume existing SQL tables and views were named in original
@@ -388,7 +375,7 @@ optionProcessor.command('manageConstraints')
 
 optionProcessor.command('toCsn')
   .option('-h, --help')
-  .option('-f, --flavor <flavor>', ['client', 'gensrc', 'universal'])
+  .option('-f, --csn-flavor <flavor>', ['client', 'gensrc', 'universal'], { aliases: ['--flavor'] })
   .option('    --with-localized')
   .help(`
   Usage: cdsc toCsn [options] <files...>
@@ -397,13 +384,13 @@ optionProcessor.command('toCsn')
 
   Options
    -h, --help             Show this help text
-   -f, --flavor <flavor>  Generate CSN in one of two flavors:
-                            client  : (default) Standard CSN consumable by clients and backends
-                            gensrc  : CSN specifically for use as a source, e.g. for
-                                      combination with additional "extend" or "annotate"
-                                      statements, but not suitable for consumption by clients or
-                                      backends
-                            universal: in development (BETA)
+   -f, --csn-flavor <flavor>  Generate CSN in one of two flavors:
+                                client  : (default) Standard CSN consumable by clients and backends
+                                gensrc  : CSN specifically for use as a source, e.g. for
+                                          combination with additional "extend" or "annotate"
+                                          statements, but not suitable for consumption by clients or
+                                          backends
+                                universal: in development (BETA)
 
   Internal options (for testing only, may be changed/removed at any time)
        --with-localized   Add localized convenience views to the CSN output.
@@ -422,6 +409,26 @@ optionProcessor.command('parseCdl')
    -h, --help             Show this help text
 `);
 
+optionProcessor.command('parseOnly')
+  .option('-h, --help')
+  .option('    --trace-parser')
+  .option('    --trace-parser-amb')
+  .positionalArgument('<file>')
+  .help(`
+  Usage: cdsc parseOnly [options] <files...>
+
+  (internal): Stop compilation after parsing and write messages to <stderr>.
+  Per default, nothing is printed.  With \`--raw-output +\`, XSN is printed
+  to <stdout>.
+
+  Options
+   -h, --help               Show this help text
+
+  Diagnostic options
+       --trace-parser       Trace parser
+       --trace-parser-amb   Trace parser ambiguities
+`);
+
 optionProcessor.command('explain')
   .option('-h, --help')
   .positionalArgument('<message-id>')

package/lib/render/.eslintrc.json

@@ -16,6 +16,6 @@
     "no-shadow": "warn"
   },
   "env": {
-    "es6": true
+    "es2020": true
   }
 }

package/lib/render/DuplicateChecker.js

@@ -2,12 +2,14 @@
  * Usage: Create an instance to process artifacts which should be checked for collision.
  * First call addArtifact to specify the current artifact,
  * then call addElement to register the elements of the current artifact.
- * Finally call the "done" function to check for duplicates.
- * In addition the internal structures will be reinitialized to enable reuse of the instance.
+ * Finally, call the "done" function to check for duplicates.
+ * In addition, the internal structures will be reinitialized to enable reuse of the instance.
  */
 
 'use strict';
 
+const { forEach } = require('../utils/objectUtils');
+
 /**
  * database name - uppercase if not quoted
  *
@@ -121,9 +123,4 @@ class DuplicateChecker {
   }
 }
 
-function forEach(obj, callback) {
-  for (const key in obj)
-    callback(key, obj[key]);
-}
-
 module.exports = DuplicateChecker;

package/lib/render/manageConstraints.js

@@ -6,11 +6,78 @@ const {
   getResultingName,
 } = require('../model/csnUtils');
 const { forEach } = require('../utils/objectUtils');
+const { makeMessageFunction } = require('../base/messages');
+const { optionProcessor } = require('../optionProcessor');
+const { transformForHanaWithCsn } = require('../transform/forHanaNew');
 
 const {
   renderReferentialConstraint, getIdentifierUtils,
 } = require('./utils/sql');
 
+/**
+ * Used only by `cdsc manageConstraints`.
+ * Not part of our API, yet.
+ *
+ * @param {CSN.Model} csn
+ * @param {CSN.Options} options
+ */
+function alterConstraintsWithCsn(csn, options) {
+  const { error } = makeMessageFunction(csn, options, 'manageConstraints');
+
+  const {
+    drop, alter, names, src, violations,
+  } = options.manageConstraints || {};
+
+  if (drop && alter)
+    error(null, null, 'Option “--drop” can\'t be combined with “--alter”');
+
+  options.sqlDialect = 'hana';
+  options.sqlMapping = names || 'plain';
+
+  // Of course we want the database constraints
+  options.assertIntegrityType = 'DB';
+
+  const transformedOptions = _transformSqlOptions(csn, options);
+  const forSqlCsn = transformForHanaWithCsn(csn, transformedOptions, 'to.sql');
+
+  if (violations && src && src !== 'sql')
+    error(null, null, `Option “--violations“ can't be combined with source style “${src}“`);
+
+  let intermediateResult;
+  if (violations)
+    intermediateResult = listReferentialIntegrityViolations(forSqlCsn, transformedOptions);
+  else
+    intermediateResult = manageConstraints(forSqlCsn, transformedOptions);
+
+  return intermediateResult;
+}
+
+function _transformSqlOptions(model, options) {
+  // Merge options with defaults.
+  options = Object.assign({ sqlMapping: 'plain', sqlDialect: 'plain' }, options);
+  options.toSql = true;
+
+  if (!options.src && !options.csn)
+    options.src = 'sql';
+
+  const { warning, error } = makeMessageFunction(model, options, 'to.sql');
+
+  optionProcessor.verifyOptions(options, 'toSql', true)
+    .forEach(complaint => warning(null, null, `${complaint}`));
+
+  if (options.sqlDialect !== 'hana') {
+    // CDXCORE-465, 'quoted' and 'hdbcds' are to be used in combination with dialect 'hana' only
+    if (options.sqlMapping === 'quoted' || options.sqlMapping === 'hdbcds')
+      error(null, null, `Option "{ sqlDialect: '${options.sqlDialect}' }" can't be combined with "{ sqlMapping: '${options.sqlMapping}' }"`);
+
+    // No non-HANA SQL for HDI
+    if (options.src === 'hdi')
+      error(null, null, `Option "{ sqlDialect: '${options.sqlDialect}' }" can't be used for HDI"`);
+  }
+
+  return options;
+}
+
 /**
  * This render middleware can be used to generate SQL DDL ALTER TABLE <table> ALTER / ADD / DROP CONSTRAINT <constraint> statements for a given CDL model.
  * Moreover, it can be used to generate .hdbconstraint artifacts.
@@ -38,7 +105,7 @@ function manageConstraints(csn, options) {
           return;
         }
         let alterTableStatement = '';
-        alterTableStatement += `${indent}ALTER TABLE ${quoteSqlId(getResultingName(csn, options.toSql.names, constraint.dependentTable))}`;
+        alterTableStatement += `${indent}ALTER TABLE ${quoteSqlId(getResultingName(csn, options.sqlMapping, constraint.dependentTable))}`;
         if (renderAlterConstraintStatement)
           alterTableStatement += `\n${indent}ALTER ${renderedConstraint};`;
         else if (drop)
@@ -172,7 +239,7 @@ function listReferentialIntegrityViolations(csn, options) {
   }
 
   function quoteAndGetResultingName(id) {
-    return quoteSqlId(getResultingName(csn, options.toSql.names, id));
+    return quoteSqlId(getResultingName(csn, options.sqlMapping, id));
   }
 
   return resultArtifacts;
@@ -192,6 +259,7 @@ function getListOfAllConstraints(csn) {
 }
 
 module.exports = {
+  alterConstraintsWithCsn,
   manageConstraints,
   listReferentialIntegrityViolations,
 };