From 47b5d7a2dfefc8d9b2c006783b93393106f32f08 Mon Sep 17 00:00:00 2001 From: Justin Greywolf Date: Fri, 5 Jun 2020 17:10:33 -0700 Subject: [PATCH] 1378-accept multiple parameters with methods sec --- package.json | 1 + src/diagrams/class/parser/classDiagram.jison | 10 ++-- src/diagrams/class/svgDraw.js | 60 +++++++++++--------- yarn.lock | 12 ++++ 4 files changed, 52 insertions(+), 31 deletions(-) diff --git a/package.json b/package.json index 90c991b46..5ca9a34b2 100644 --- a/package.json +++ b/package.json @@ -59,6 +59,7 @@ "he": "^1.2.0", "minify": "^4.1.1", "moment-mini": "^2.22.1", + "node": "^14.3.0", "scope-css": "^1.2.1" }, "devDependencies": { diff --git a/src/diagrams/class/parser/classDiagram.jison b/src/diagrams/class/parser/classDiagram.jison index 12e9a2564..bd8cf6f59 100644 --- a/src/diagrams/class/parser/classDiagram.jison +++ b/src/diagrams/class/parser/classDiagram.jison @@ -10,15 +10,15 @@ %% \%\%[^\n]*\n* /* do nothing */ -\n+ return 'NEWLINE'; +\n+ return 'NEWLINE'; \s+ /* skip whitespace */ "classDiagram" return 'CLASS_DIAGRAM'; [\{] { this.begin("struct"); /*console.log('Starting struct');*/return 'STRUCT_START';} <> return "EOF_IN_STRUCT"; [\{] return "OPEN_IN_STRUCT"; -\} { /*console.log('Ending struct');*/this.popState(); return 'STRUCT_STOP';}} +\} { /*console.log('Ending struct');*/this.popState(); return 'STRUCT_STOP';}} [\n] /* nothing */ -[^\{\}\n]* { /*console.log('lex-member: ' + yytext);*/ return "MEMBER";} +[^\{\}\n]* { /*console.log('lex-member: ' + yytext);*/ return "MEMBER";} @@ -40,7 +40,7 @@ \s*\|\> return 'EXTENSION'; \s*\> return 'DEPENDENCY'; \s*\< return 'DEPENDENCY'; -\s*\* return 'COMPOSITION'; +\s*\* return 'COMPOSITION'; \s*o return 'AGGREGATION'; \-\- return 'LINE'; \.\. return 'DOTTED_LINE'; @@ -53,7 +53,7 @@ \= return 'EQUALS'; \w+ return 'ALPHA'; [!"#$%&'*+,-.`?\\/] return 'PUNCTUATION'; -[0-9]+ return 'NUM'; +[0-9]+ return 'NUM'; [\u00AA\u00B5\u00BA\u00C0-\u00D6\u00D8-\u00F6]| [\u00F8-\u02C1\u02C6-\u02D1\u02E0-\u02E4\u02EC\u02EE\u0370-\u0374\u0376\u0377]| [\u037A-\u037D\u0386\u0388-\u038A\u038C\u038E-\u03A1\u03A3-\u03F5]| diff --git a/src/diagrams/class/svgDraw.js b/src/diagrams/class/svgDraw.js index a93cd1e87..c97259a55 100644 --- a/src/diagrams/class/svgDraw.js +++ b/src/diagrams/class/svgDraw.js @@ -278,8 +278,8 @@ export const drawClass = function(elem, classDef, conf) { }; export const parseMember = function(text) { - const fieldRegEx = /^(\+|-|~|#)?(\w+)(~\w+~|\[\])?\s+(\w+)$/; - const methodRegEx = /^(\+|-|~|#)?(\w+)\s?\(\s*(\w+(~\w+~|\[\])?\s*(\w+)?)?\s*\)\s?([*|$])?\s?(\w+(~\w+~|\[\])?)?\s*$/; + const fieldRegEx = /^(\+|-|~|#)?(\w+)(~\w+~|\[\])?\s+(\w+)/; + const methodRegEx = /^(\+|-|~|#)?(\w+)\((.*)\)(\*|\$)?\s?(.*)?/; let fieldMatch = text.match(fieldRegEx); let methodMatch = text.match(methodRegEx); @@ -294,43 +294,53 @@ export const parseMember = function(text) { }; const buildFieldDisplay = function(parsedText) { - let visibility = parsedText[1] ? parsedText[1].trim() : ''; - let fieldType = parsedText[2] ? parsedText[2].trim() : ''; - let genericType = parsedText[3] ? parseGenericTypes(parsedText[3]) : ''; - let fieldName = parsedText[4] ? parsedText[4].trim() : ''; + let displayText = ''; + + try { + let visibility = parsedText[1] ? parsedText[1].trim() : ''; + let fieldType = parsedText[2] ? parseGenericTypes(parsedText[2]) : ''; + let fieldName = parsedText[3] ? parsedText[3].trim() : ''; + + displayText = visibility + fieldType + ' ' + fieldName; + } catch (err) { + displayText = parsedText; + } return { - displayText: visibility + fieldType + genericType + ' ' + fieldName, + displayText: displayText, cssStyle: '' }; }; const buildMethodDisplay = function(parsedText) { let cssStyle = ''; - let displayText = parsedText; + let displayText = ''; - let visibility = parsedText[1] ? parsedText[1].trim() : ''; - let methodName = parsedText[2] ? parsedText[2].trim() : ''; - let parameters = parsedText[3] ? parseGenericTypes(parsedText[3]) : ''; - let classifier = parsedText[6] ? parsedText[6].trim() : ''; - let returnType = parsedText[7] ? ' : ' + parseGenericTypes(parsedText[7]).trim() : ''; + try { + let visibility = parsedText[1] ? parsedText[1].trim() : ''; + let methodName = parsedText[2] ? parsedText[2].trim() : ''; + let parameters = parsedText[3] ? parseGenericTypes(parsedText[3]) : ''; + let classifier = parsedText[4] ? parsedText[4].trim() : ''; + let returnType = parsedText[5] ? ' : ' + parseGenericTypes(parsedText[5]).trim() : ''; - displayText = visibility + methodName + '(' + parameters + ')' + returnType; + displayText = visibility + methodName + '(' + parameters + ')' + returnType; - cssStyle = parseClassifier(classifier); + cssStyle = parseClassifier(classifier); + } catch (err) { + displayText = parsedText; + } - let member = { + return { displayText: displayText, cssStyle: cssStyle }; - - return member; }; const buildLegacyDisplay = function(text) { // if for some reason we dont have any match, use old format to parse text - let memberText = ''; + let displayText = ''; let cssStyle = ''; + let memberText = ''; let returnType = ''; let methodStart = text.indexOf('('); let methodEnd = text.indexOf(')'); @@ -338,12 +348,12 @@ const buildLegacyDisplay = function(text) { if (methodStart > 1 && methodEnd > methodStart && methodEnd <= text.length) { let parsedText = text.match(/(\+|-|~|#)?(\w+)/); let visibility = parsedText[1] ? parsedText[1].trim() : ''; - let methodName = parsedText[2]; + let methodName = text.substring(0, methodStart); let parameters = text.substring(methodStart + 1, methodEnd); let classifier = text.substring(methodEnd, methodEnd + 1); cssStyle = parseClassifier(classifier); - memberText = visibility + methodName + '(' + parseGenericTypes(parameters.trim()) + ')'; + displayText = visibility + methodName + '(' + parseGenericTypes(parameters.trim()) + ')'; if (methodEnd < memberText.length) { returnType = text.substring(methodEnd + 2).trim(); @@ -353,15 +363,13 @@ const buildLegacyDisplay = function(text) { } } else { // finally - if all else fails, just send the text back as written (other than parsing for generic types) - memberText = parseGenericTypes(text); + displayText = parseGenericTypes(text); } - let member = { - displayText: memberText + returnType, + return { + displayText: displayText, cssStyle: cssStyle }; - - return member; }; const addTspan = function(textEl, txt, isFirst, conf) { diff --git a/yarn.lock b/yarn.lock index 5b27d9a91..320906b14 100644 --- a/yarn.lock +++ b/yarn.lock @@ -7548,6 +7548,11 @@ no-case@^2.2.0: dependencies: lower-case "^1.1.1" +node-bin-setup@^1.0.0: + version "1.0.6" + resolved "https://registry.yarnpkg.com/node-bin-setup/-/node-bin-setup-1.0.6.tgz#4b5c9bb937ece702d7069b36ca78af4684677528" + integrity sha512-uPIxXNis1CRbv1DwqAxkgBk5NFV3s7cMN/Gf556jSw6jBvV7ca4F9lRL/8cALcZecRibeqU+5dFYqFFmzv5a0Q== + node-forge@0.9.0: version "0.9.0" resolved "https://registry.yarnpkg.com/node-forge/-/node-forge-0.9.0.tgz#d624050edbb44874adca12bb9a52ec63cb782579" @@ -7651,6 +7656,13 @@ node-sass@^4.12.0: stdout-stream "^1.4.0" "true-case-path" "^1.0.2" +node@^14.3.0: + version "14.3.0" + resolved "https://registry.yarnpkg.com/node/-/node-14.3.0.tgz#a0aa79f74add23cf84a2d3ac9d631bbed3ca496a" + integrity sha512-wYdyAhpzQ+iaXnDby6JUok/efke+TW2VJk8lOFc4B6F1omIiBG6gt/xVG074btcliY9Sw49D2veah8bZ20lGEA== + dependencies: + node-bin-setup "^1.0.0" + nomnom@1.5.2: version "1.5.2" resolved "https://registry.yarnpkg.com/nomnom/-/nomnom-1.5.2.tgz#f4345448a853cfbd5c0d26320f2477ab0526fe2f"