Merge pull request #38 from TakayukiHoshi1984/modify_codegen_1_8_2

DeviceConnectCodegen 1.8.2

Author: TakayukiHoshi1984

DeviceConnectCodegen/README.md

@@ -33,7 +33,7 @@ DeviceConnect Codegenは、DeviceConnectシステム上で動作するプラグ
 |samples/profiles-specs|シェルスクリプトのサンプルに入力するプロファイル定義ファイル群。|
 
 ## Get Started
-[deviceconnect-codegen-project-1.7.0-dist.zip](https://github.com/TakayukiHoshi1984/DeviceConnect-Experiments/releases/tag/codegen-v1.7.0) をPC上の任意の場所にダウンロードし、解凍してください。
+[deviceconnect-codegen-project-1.8.2-dist.zip](https://github.com/TakayukiHoshi1984/DeviceConnect-Experiments/releases/tag/codegen-v1.8.2) をPC上の任意の場所にダウンロードし、解凍してください。
 
 解凍後、ターミナルを起動し、以下のコマンドによりをサンプルのスケルトンコードを生成してください。
 

DeviceConnectCodegen/RELEASENOTE.TXT

@@ -1,4 +1,11 @@
 # 変更履歴
+2018/12/10: v1.8.2 [共通] swagger-codegen v2.2.3 を使用するように変更。v2.2.1 に関する脆弱性への対処のため。
+                   [Markdown] 不具合修正: GitHub Wikiにアップロードすると各API仕様へのタグジャンプが行えない。
+
+2018/09/27: v1.8.1 [NodeJS] メンテナンス: NodeJS向けに出力するデバイスプラグインのREADMEにServiceInformationを追加
+
+2018/09/12: v1.8.0 [NodeJS] NodeJS向けに出力するデバイスプラグインのソースコードを改善
+
 2018/03/01: v1.6.0 [Android] オプション追加: --gradle-plugin-version, --sdk, --signing-config, --template-dir
                    [共通] 不具合修正: basePathプロパティでプロファイル名が定義されている場合に異常終了
 

DeviceConnectCodegen/modules/deviceconnect-codegen/pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>org.deviceconnect</groupId>
     <artifactId>deviceconnect-codegen</artifactId>
-    <version>1.8.1</version>
+    <version>1.8.2</version>
 
     <packaging>jar</packaging>
 
@@ -70,7 +70,7 @@
         <dependency>
             <groupId>io.swagger</groupId>
             <artifactId>swagger-codegen</artifactId>
-            <version>2.2.1</version>
+            <version>2.2.3</version>
         </dependency>
         <dependency>
             <groupId>junit</groupId>

DeviceConnectCodegen/modules/deviceconnect-codegen/src/main/java/org/deviceconnect/codegen/AbstractCodegenConfig.java

@@ -22,6 +22,8 @@
 
 public abstract class AbstractCodegenConfig extends DefaultCodegen implements DConnectCodegenConfig {
 
+    protected Swagger originalSwagger;
+
     protected Map<String, Swagger> profileSpecs;
 
     protected abstract String profileFileFolder();
@@ -41,11 +43,26 @@ public void setProfileSpecs(final Map<String, Swagger> profileSpecs) {
         this.profileSpecs = profileSpecs;
     }
 
+    @Override
+    public Swagger getOriginalSwagger() {
+        return this.originalSwagger;
+    }
+
+    @Override
+    public void setOriginalSwagger(final Swagger swagger) {
+        this.originalSwagger = swagger;
+    }
+
     @Override
     public String escapeUnsafeCharacters(final String input) {
         return input.replace("*/", "*_/").replace("/*", "/_*");
     }
 
+    @Override
+    public String escapeQuotationMark(final String input) {
+        return input.replace("\"", "\\\"");
+    }
+
     protected static String toUpperCapital(final String str, final boolean onlyFirstChar) {
         StringBuffer buf = new StringBuffer(str.length());
         buf.append(str.substring(0, 1).toUpperCase());

DeviceConnectCodegen/modules/deviceconnect-codegen/src/main/java/org/deviceconnect/codegen/DConnectCodegen.java

@@ -23,7 +23,9 @@
 import org.deviceconnect.codegen.app.HtmlAppCodegenConfig;
 import org.deviceconnect.codegen.docs.HtmlDocsCodegenConfig;
 import org.deviceconnect.codegen.docs.MarkdownDocsCodegenConfig;
+import org.deviceconnect.codegen.util.SortedSwagger;
 import org.deviceconnect.codegen.util.SwaggerJsonValidator;
+import org.deviceconnect.codegen.util.SwaggerUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -203,9 +205,11 @@ public static void main(String[] args) {
             new Codegen() {
                 @Override
                 public File writeToFile(final String filename, final String contents) throws IOException {
-                    // LICENSE ファイルは出力させない
+                    // 不要なファイルは出力させない
                     if (filename != null) {
-                        if (filename.endsWith("LICENSE") || filename.endsWith(".swagger-codegen-ignore")) {
+                        if (filename.endsWith("VERSION")
+                            || filename.endsWith("LICENSE")
+                            || filename.endsWith(".swagger-codegen-ignore")) {
                             return null;
                         }
                     }
@@ -264,6 +268,7 @@ private static void parseSwaggerFromFile(File file,
             profile.setPaths(subPaths);
         }
         config.setProfileSpecs(profiles);
+        config.setOriginalSwagger(SwaggerUtils.cloneSwagger(swagger));
     }
 
     private static void parseSwaggerFromDirectory(File dir,
@@ -301,7 +306,9 @@ public boolean accept(final File dir, final String name) {
             checkProfileName(config, profileName);
         }
         config.setProfileSpecs(profileSpecs);
-        clientOptInput.swagger(mergeSwaggers(profileSpecs));
+        Swagger swagger = mergeSwaggers(profileSpecs);
+        config.setOriginalSwagger(SwaggerUtils.cloneSwagger(swagger));
+        clientOptInput.swagger(swagger);
     }
 
     private static boolean checkSwagger(final File file) throws IOException, ProcessingException {
@@ -355,7 +362,7 @@ private static void printDuplicatedPathError(final DuplicatedPathException e) {
     }
 
     private static Swagger createProfileSpec(final Swagger swagger) {
-        Swagger profile = new Swagger();
+        Swagger profile = new SortedSwagger();
         profile.setSwagger(swagger.getSwagger());
         Info info = new Info();
         info.setTitle(swagger.getInfo().getTitle());
@@ -453,7 +460,7 @@ private static String parseProfileNameFromFileName(final String fileName) {
     }
 
     private static Swagger mergeSwaggers(Map<String, Swagger> swaggerMap) {
-        Swagger merged = new Swagger();
+        Swagger merged = new SortedSwagger();
         merged.setBasePath("/");
 
         // info
@@ -462,6 +469,21 @@ private static Swagger mergeSwaggers(Map<String, Swagger> swaggerMap) {
         info.setVersion("1.0.0");
         merged.setInfo(info);
 
+        // consumes
+        List<String> allConsumes = new ArrayList<>();
+        for (Map.Entry<String, Swagger> entry : swaggerMap.entrySet()) {
+            Swagger swagger = entry.getValue();
+            List<String> consumes = swagger.getConsumes();
+            if (consumes != null) {
+                for (String mimeType : consumes) {
+                    if (!allConsumes.contains(mimeType)) {
+                        allConsumes.add(mimeType);
+                    }
+                }
+            }
+        }
+        merged.setConsumes(allConsumes);
+
         // paths
         Map<String, Path> paths = new HashMap<>();
         for (Map.Entry<String, Swagger> swagger : swaggerMap.entrySet()) {

DeviceConnectCodegen/modules/deviceconnect-codegen/src/main/java/org/deviceconnect/codegen/DConnectCodegenConfig.java

@@ -12,7 +12,6 @@
 import io.swagger.models.Swagger;
 import org.apache.commons.cli.CommandLine;
 
-import java.io.File;
 import java.util.Map;
 
 public interface DConnectCodegenConfig extends CodegenConfig {
@@ -21,6 +20,10 @@ public interface DConnectCodegenConfig extends CodegenConfig {
 
     void setProfileSpecs(Map<String, Swagger> profileSpecs);
 
+    Swagger getOriginalSwagger();
+
+    void setOriginalSwagger(Swagger swagger);
+
     String getDefaultDisplayName();
 
     ValidationResultSet validateOptions(CommandLine cmd, ClientOpts clientOpts);

DeviceConnectCodegen/modules/deviceconnect-codegen/src/main/java/org/deviceconnect/codegen/DConnectPath.java

@@ -144,4 +144,16 @@ public String toCanonicalPathName() {
     public String toString() {
         return getPath();
     }
+
+    @Override
+    public boolean equals(final Object o) {
+        if (o == null) {
+            return false;
+        }
+        if (!(o instanceof DConnectPath)) {
+            return false;
+        }
+        DConnectPath that = (DConnectPath) o;
+        return that.pathExp.equalsIgnoreCase(this.pathExp);
+    }
 }

DeviceConnectCodegen/modules/deviceconnect-codegen/src/main/java/org/deviceconnect/codegen/MultipleSwaggerConverter.java

@@ -11,6 +11,7 @@
 import io.swagger.models.Model;
 import io.swagger.models.Path;
 import io.swagger.models.Swagger;
+import org.deviceconnect.codegen.util.SortedSwagger;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -52,7 +53,7 @@ private void convert(final Swagger swagger,
             String pathName = entry.getKey();
             DConnectPath path = DConnectPath.parsePath(basePath, pathName);
             counter.count(path.toCanonicalPathName());
-            LOGGER.info("Base Path: " + path.getBathPath() + ", Sub Path: " + path.getSubPath() + ", Profile: " + path.getProfileName());
+            LOGGER.debug("Base Path: " + path.getBathPath() + ", Sub Path: " + path.getSubPath() + ", Profile: " + path.getProfileName());
 
             String key = path.getProfileName();
             Swagger cache = result.get(key);
@@ -87,7 +88,7 @@ private void convert(final Swagger swagger,
     }
 
     private static Swagger createProfileSpec(final Swagger swagger) {
-        Swagger profile = new Swagger();
+        Swagger profile = new SortedSwagger();
         profile.setSwagger(swagger.getSwagger());
         Info info = new Info();
         info.setTitle(swagger.getInfo().getTitle());

DeviceConnectCodegen/modules/deviceconnect-codegen/src/main/java/org/deviceconnect/codegen/app/HtmlAppCodegenConfig.java

@@ -15,17 +15,22 @@
 import io.swagger.codegen.DefaultCodegen;
 import io.swagger.codegen.SupportingFile;
 import io.swagger.models.Swagger;
-import io.swagger.parser.SwaggerParser;
 import org.apache.commons.cli.CommandLine;
+import org.deviceconnect.codegen.AbstractCodegenConfig;
 import org.deviceconnect.codegen.DConnectCodegenConfig;
 import org.deviceconnect.codegen.ValidationResultSet;
 
 import java.util.Map;
 
-public class HtmlAppCodegenConfig extends DefaultCodegen implements DConnectCodegenConfig {
+public class HtmlAppCodegenConfig extends AbstractCodegenConfig implements DConnectCodegenConfig {
 
     private Map<String, Swagger> profileSpecs;
 
+    @Override
+    protected String profileFileFolder() {
+        return null; // Not be used.
+    }
+
     @Override
     public ValidationResultSet validateOptions(final CommandLine cmd, final ClientOpts clientOpts) {
         return new ValidationResultSet();

DeviceConnectCodegen/modules/deviceconnect-codegen/src/main/java/org/deviceconnect/codegen/docs/AbstractDocsCodegenConfig.java

@@ -68,7 +68,6 @@ public void processOpts() {
                 basePath = "/gotapi/" + profileKey;
                 profileName = profileKey;
             } else {
-                System.out.println("basePath: " + basePath);
                 profileName = basePath.split("/")[2];
             }
 
@@ -149,6 +148,13 @@ List<Object> paramList() {
                         }
                         Object response = createResponseDocument(profileSpec, op);
                         Object event = createEventDocument(profileSpec, op);
+                        String tag() {
+                            String tag = this.name;
+                            tag = tag.toLowerCase();
+                            tag = tag.replaceAll(" ", "-");  // 半角スペース -> 半角ハイフン
+                            tag = tag.replaceAll("/", "");  // 半角スラッシュ -> 空文字
+                            return tag;
+                        }
                     });
                 }
             }
@@ -278,7 +284,7 @@ private Object createResponseDocument(final Swagger swagger, final Operation ope
     }
 
     private Object createEventDocument(final Swagger swagger, final Operation operation) {
-        DConnectOperation dConnectOperation = DConnectOperation.parse(swagger, operation);
+        DConnectOperation dConnectOperation = DConnectOperation.parse(operation);
         if (dConnectOperation == null) {
             return null;
         }