Topic/miurahr/java17/re organize source tree #3
Conversation
Signed-off-by: Hiroshi Miura <[email protected]>
HTMLUtils methods are used in MosesTranslate. This make dependency from MT connector to Filter driver. This indicate the class is common utility so refactor the class into org.omegat.util package. - Rename it to org.omegat.util.HtmlUtils - Deprecate org.omegat.filters2.html2.HTMLUtils class and refer to HtmlUtils for compatibility. Signed-off-by: Hiroshi Miura <[email protected]>
TagValidationTool class depends on PoFilter class to detect it is called from PoFilter. A change drop import and check with class name rather than class itself. Signed-off-by: Hiroshi Miura <[email protected]>
Signed-off-by: Hiroshi Miura <[email protected]>
- Update TMXWriterTest to check with system line separator It is because XML parser can automatically change /r/n into XML entity 
/n when System.lineSeparator() is /n - Update TMXReader2 to detect both "lang" and "xml:lang" some implementation return "lang" from "xml:lang" but other can return "xml:lang" as is. - as a workaround for FasterXML/jackson-modules-base#127 Add @JacksonXmlElementWrapper(useWrapping=false) for mask element for the parser of omegat.project file. Signed-off-by: Hiroshi Miura <[email protected]>
- Ref: StAX parser in DGT-OmegaT Signed-off-by: Hiroshi Miura <[email protected]>
Signed-off-by: Hiroshi Miura <[email protected]>
Use OmegaT-DGT's SRX loader Signed-off-by: Hiroshi Miura <[email protected]>
miurahr
force-pushed
the
topic/miurahr/java17/re-organize-source-tree
branch
2 times, most recently
from
a071c5f to
1d19107
Compare
|
|
miurahr
force-pushed
the
topic/miurahr/java17/re-organize-source-tree
branch
6 times, most recently
from
184ffb8 to
3bab399
Compare
Signed-off-by: Hiroshi Miura <[email protected]>
- Support java17 - Support module Signed-off-by: Hiroshi Miura <[email protected]>
miurahr
force-pushed
the
topic/miurahr/java17/re-organize-source-tree
branch
from
3bab399 to
913a5d9
Compare
![sonatype-lift[bot]](https://avatars.githubusercontent.com/in/9843?s=60&v=4){kind=small}
| @@ -202,9 +204,10 @@ dependencies { | |||
| // Moses MT connector | |||
| implementation 'org.apache.xmlrpc:xmlrpc-client:3.1.3' | |||
There was a problem hiding this comment.
Critical Vulnerability:
pkg:maven/org.apache.xmlrpc/[email protected]
1 Critical, 1 High, 1 Medium, 0 Low, 0 None vulnerabilities have been found across 2 dependencies
To see more details about this component, go to the Sonatype Lift console
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
| implementation "com.fasterxml.jackson.core:jackson-core:2.12.3" | ||
| implementation "com.fasterxml.jackson.core:jackson-databind:2.12.3" | ||
| implementation "com.fasterxml.jackson.dataformat:jackson-dataformat-xml:2.12.3" |
There was a problem hiding this comment.
High Vulnerability:
pkg:maven/com.fasterxml.jackson.dataformat/[email protected]
0 Critical, 4 High, 0 Medium, 0 Low, 0 None vulnerabilities have been found across 1 dependencies
To see more details about this component, go to the Sonatype Lift console
ℹ️ Learn about @sonatype-lift commands
You can reply with the following commands. For example, reply with @sonatype-lift ignoreall to leave out all findings.
| Command | Usage |
|---|---|
@sonatype-lift ignore |
Leave out the above finding from this PR |
@sonatype-lift ignoreall |
Leave out all the existing findings from this PR |
@sonatype-lift exclude <file|issue|path|tool> |
Exclude specified file|issue|path|tool from Lift findings by updating your config.toml file |
Note: When talking to LiftBot, you need to refresh the page to see its response.
Click here to add LiftBot to another repo.
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
Pull request type
Please mark github LABEL of the type of change your PR introduces:
Which ticket is resolved?
Title ex. fix...
Title
What does this PR change?
Other information