feat: add ParseFile wrapping parser.Parse(uri string) (#15)

Author: bfabio

go-src/publiccode-parser-wrapper.go

@@ -49,6 +49,90 @@ func NewParser(disableNetwork C.bool, branch *C.char, baseURL *C.char) C.ParserH
 	return C.ParserHandle(cgo.NewHandle(p))
 }
 
+//export ParseFile
+func ParseFile(handle C.ParserHandle, uri *C.char) *C.struct_ParseResult {
+	result := (*C.struct_ParseResult)(C.calloc(1, C.size_t(C.sizeof_struct_ParseResult)))
+	result.Data = nil
+	result.Error = nil
+	result.ErrorCount = 0
+	result.Errors = nil
+	result.WarningCount = 0
+	result.Warnings = nil
+
+	parser, err := toGoParser(handle)
+	if err != nil {
+		result.Error = C.CString("Failed create a Parser: " + err.Error())
+
+		return result
+	}
+
+	goString := C.GoString(uri)
+
+	pc, err := parser.Parse(goString)
+
+	if err != nil {
+		if validationRes, ok := err.(publiccode.ValidationResults); ok {
+			for _, res := range validationRes {
+				switch res.(type) {
+				case publiccode.ValidationError:
+					result.ErrorCount += 1
+				case publiccode.ValidationWarning:
+					result.WarningCount += 1
+				}
+			}
+
+			var errorsSlice []*C.char
+			cErrors := unsafe.Pointer(nil)
+			if result.ErrorCount > 0 {
+				cErrors = C.malloc(C.size_t(result.ErrorCount) * C.size_t(unsafe.Sizeof(uintptr(0))))
+				errorsSlice = (*[1 << 28]*C.char)(cErrors)[:result.ErrorCount:result.ErrorCount]
+			}
+
+			var warningsSlice []*C.char
+			cWarnings := unsafe.Pointer(nil)
+			if result.WarningCount > 0 {
+				cWarnings = C.malloc(C.size_t(result.WarningCount) * C.size_t(unsafe.Sizeof(uintptr(0))))
+				warningsSlice = (*[1 << 28]*C.char)(cWarnings)[:result.WarningCount:result.WarningCount]
+			}
+
+			errIdx := 0
+			warnIdx := 0
+			for _, res := range validationRes {
+				switch res.(type) {
+				case publiccode.ValidationError:
+					errorsSlice[errIdx] = C.CString(res.Error())
+					errIdx += 1
+				case publiccode.ValidationWarning:
+					warningsSlice[warnIdx] = C.CString(res.Error())
+					warnIdx += 1
+				}
+			}
+
+			result.Errors = (**C.char)(cErrors)
+			result.Warnings = (**C.char)(cWarnings)
+		} else {
+			result.Error = C.CString(err.Error())
+
+			return result
+		}
+
+		if result.ErrorCount > 0 {
+			return result
+		}
+	}
+
+	jsonData, err := json.Marshal(pc)
+	if err != nil {
+		result.Error = C.CString("Failed to marshal result to JSON: " + err.Error())
+
+		return result
+	}
+
+	result.Data = C.CString(string(jsonData))
+
+	return result
+}
+
 //export ParseString
 func ParseString(handle C.ParserHandle, content *C.char) *C.struct_ParseResult {
 	result := (*C.struct_ParseResult)(C.calloc(1, C.size_t(C.sizeof_struct_ParseResult)))

lib/libpubliccode-parser.h

@@ -99,6 +99,7 @@ extern "C" {
 #endif
 
 extern ParserHandle NewParser(_Bool disableNetwork, char* branch, char* baseURL);
+extern struct ParseResult* ParseFile(ParserHandle handle, char* uri);
 extern struct ParseResult* ParseString(ParserHandle handle, char* content);
 extern void FreeResult(struct ParseResult* result);
 extern void FreeParser(ParserHandle handle);

src/Parser.php

@@ -70,25 +70,14 @@ public function parse(string $content): PublicCode
      */
     public function parseFile(string $filePath): PublicCode
     {
-        if (!file_exists($filePath)) {
-            throw new ParserException("File not found: {$filePath}");
-        }
+        /** @phpstan-ignore-next-line */
+        $result = $this->ffi->ParseFile($this->handle, $filePath);
 
-        $fp = @fopen($filePath, 'rb');
-        if ($fp === false) {
-            throw new ParserException("Cannot open file: {$filePath}");
+        if ($result === null) {
+            throw new ParserException("Failed to parse $filePath");
         }
 
-        try {
-            $content = stream_get_contents($fp);
-            if ($content === false) {
-                throw new ParserException('Failed to read from stream');
-            }
-
-            return $this->parse($content);
-        } finally {
-            fclose($fp);
-        }
+        return $this->processResult($result);
     }
 
     /**
@@ -135,6 +124,7 @@ private function getFFI(): FFI
 
 	ParserHandle NewParser(bool disableNetwork, const char* branch, const char* baseURL);
 	ParseResult* ParseString(ParserHandle handle, const char* content);
+	ParseResult* ParseFile(ParserHandle handle, const char* uri);
 	void FreeResult(ParseResult* result);
 	void FreeParser(ParserHandle handle);
 	CDEF;