search

jhipster / generator-jhipster

JHipster is a development platform to quickly generate, develop, & deploy modern web applications & microservice architectures.
https://www.jhipster.tech
Apache License 2.0
21.53k stars 4.02k forks source link

JDL file name shall be reported in an error message for JDL import #23533

Closed vw98075 closed 3 months ago

vw98075 commented 1 year ago
Overview of the issue

If an error occurs while importing multiple JDL files, a user won't know which file the error occurs from an error message such as 

error: Syntax error message:
        MismatchedTokenException: Found an invalid token '@', at line: 559 and column: 48.
        Please make sure your JDL content does not use invalid characters, keywords or options.
Motivation for or Use Case

A better use experience by making it easier to locate a bug.

Reproduce the error
Related issues
Suggest a Fix
JHipster Version(s)

7.9.4

JHipster configuration
Entity configuration(s) entityName.json files generated in the .jhipster directory
Browsers and Operating System
mshima commented 1 year ago

@vw98075 can you provide a PR?

vw98075 commented 1 year ago

Once I am released from my current project, I can look into this bug although I am not a Javascript person. That will take three months or so.

qmonmert commented 10 months ago

we have the file name in logs

image

here jhipster-jdl.jdl

cc @mshima @DanielFran

mshima commented 10 months ago

The issue is about parsing more than one jdl. All jdl files are concatenated and treated as 1. To implement this enhancement, we need to add markers to map lines to a jdl filename.

github-actions[bot] commented 4 months ago

This issue is stale because it has been open for too long without any activity. Due to the moving nature of jhipster generated application, bugs can become invalid. If this issue still applies please comment otherwise it will be closed in 7 days