fix: duplication class name with ignore case.

.github/workflows/ci.yml

@@ -0,0 +1,28 @@
+# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
+
+name: Node.js CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [10.x, 12.x, 14.x]
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v1
+      with:
+        node-version: ${{ matrix.node-version }}
+    - run: npm install
+    - run: npm run ci

src/langs/php/combinator.js

@@ -101,16 +101,16 @@ class Combinator extends CombinatorBase {
       let tmp = realFullClassName.split('\\');
       last = tmp[tmp.length - 1];
     }
-
-    if (this.classNameMap[last]) {
-      if (this.classNameMap[last] !== realFullClassName) {
+    let lower = last.toLowerCase();
+    if (this.classNameMap[lower]) {
+      if (this.classNameMap[lower] !== realFullClassName) {
         // return full class name if already have same name class
         return realFullClassName;
       }
       return last;
     }
 
-    this.classNameMap[last] = realFullClassName;
+    this.classNameMap[lower] = realFullClassName;
     if (this.thirdPackageClientAlias[className]) {
       // has alias
       this.includeList.push({ import: realFullClassName, alias: this.thirdPackageClientAlias[className] });
@@ -145,14 +145,15 @@ class Combinator extends CombinatorBase {
 
     let tmp = realFullClassName.split('\\');
     let last = tmp[tmp.length - 1];
-    if (this.classNameMap[last]) {
-      if (this.classNameMap[last] !== realFullClassName) {
+    let lower = last.toLowerCase();
+    if (this.classNameMap[lower]) {
+      if (this.classNameMap[lower] !== realFullClassName) {
         // return full class name if already have same name class
         return realFullClassName;
       }
       return last;
     }
-    this.classNameMap[last] = realFullClassName;
+    this.classNameMap[lower] = realFullClassName;
     this.includeModelList.push({ import: realFullClassName, alias: null });
     return last;
   }
@@ -327,13 +328,15 @@ class Combinator extends CombinatorBase {
       className = this.config.client.name;
       this.config.filename = className;
     }
+    className = _avoidKeywords(className);
     if (object.annotations.length > 0) {
       this.emitAnnotations(emitter, object.annotations);
     }
     if (_isKeywords(className)) {
-      this.config.filename = _avoidKeywords(className);
+      this.config.filename = className;
     }
-    emitter.emitln(`class ${_avoidKeywords(className)} ${parent}{`, this.level);
+    this.classNameMap[className.toLowerCase()] = className;
+    emitter.emitln(`class ${className} ${parent}{`, this.level);
     this.levelUp();
     const notes = this.resolveNotes(object.body);
     if (Object.keys(notes).length > 0) {

tests/expected/complex/Models/ComplexRequest.php

@@ -18,6 +18,7 @@ class ComplexRequest extends Model {
         'part' => 'Part',
     ];
     public function validate() {
+        Model::validateRequired('duplicatName', $this->duplicatName, true);
         Model::validateRequired('accessKey', $this->accessKey, true);
         Model::validateRequired('body', $this->body, true);
         Model::validateRequired('strs', $this->strs, true);
@@ -27,6 +28,9 @@ public function validate() {
     }
     public function toMap() {
         $res = [];
+        if (null !== $this->duplicatName) {
+            $res['duplicatName'] = null !== $this->duplicatName ? $this->duplicatName->toMap() : null;
+        }
         if (null !== $this->accessKey) {
             $res['accessKey'] = $this->accessKey;
         }
@@ -62,6 +66,9 @@ public function toMap() {
      */
     public static function fromMap($map = []) {
         $model = new self();
+        if(isset($map['duplicatName'])){
+            $model->duplicatName = \Source\Models\complexrequest::fromMap($map['duplicatName']);
+        }
         if(isset($map['accessKey'])){
             $model->accessKey = $map['accessKey'];
         }
@@ -93,6 +100,11 @@ public static function fromMap($map = []) {
         }
         return $model;
     }
+    /**
+     * @var \Source\Models\complexrequest
+     */
+    public $duplicatName;
+
     /**
      * @var string
      */

tests/expected/model/Models/Class_.php

@@ -13,7 +13,7 @@ public function toMap() {
     }
     /**
      * @param array $map
-     * @return Class
+     * @return Class_
      */
     public static function fromMap($map = []) {
         $model = new self();

tests/fixtures/complex/libraries/import.dara

@@ -30,3 +30,7 @@ model Request = {
     extra: map[string] string
   },
 }
+
+model complexrequest = {
+
+}

tests/fixtures/complex/main.dara

@@ -12,6 +12,7 @@ init(config: Source.Config){
 }
 
 model ComplexRequest = {
+  duplicatName: Source.complexrequest,
   accessKey: string,
   body: readable(name='Body', example='Body', description='Body'),
   strs: [ string ](name='Strs', example='Strs', description='Strs'),