-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed some SonarCloud issues from the complete code analysis #49
Conversation
…ccording to SonarCloud suggestions; unfortunately, I broke the JAR execution with this because the resource files are not found anymore when executed from JAR, so WIP;
…rom JAR again now; added IConfiguration interface; added configurations as param in constructor of MainViewController and AboutViewController;
…tion properties file;
…and not de novo constructed;
* production: Fix javadoc Add try-catch-block to sorting method Refine javadoc Add javadoc Fix Spotless requirements Optimize imports Change SortType to boolean Add enums to TableViews Add javadoc Add enum for data model properties Fix javadoc Change sort type to enum of TableColumn # Conflicts: # src/main/java/de/unijena/cheminf/mortar/controller/HistogramViewController.java
…into a separate method;
…ping to molecule in molecules tab enabled;
…ewToolsManager, making fields in MainMenuBar and FragmentsdataTableView final, adjusting some regex patterns, some manual linting, more documentation on how reflection is used to fill the data tables;
@@ -252,7 +251,7 @@ public static void guiBindControlSizeToParentPane(Pane aParentPane, Control aChi | |||
* @return GUI input pattern for integer values | |||
*/ | |||
public static Pattern getIntegerPattern(){ | |||
return Pattern.compile("-?(([1-9][0-9]*)|0)?"); | |||
return Pattern.compile("-?(([1-9]\\d*)|0)?"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Be careful with this, [0-9] is not always equivalent to \d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for noticing! I think I'm going to reverse this anyway since [0-9] is more straightforward for people not dealing with regex every day (i.e. me and potential students)...
…in the background);
…introducing <?> operator to define undefined generic type of properties;
…constant defining the character;
…c display name of an enum constant; prepared the property-using classes like fragmenters, settings container, and view tools for its usage, i.e. no enum.valueof() calls anymore;
…wraps an enum constant with associated display name and tooltip to be used in the GUI; started replacing SimpleEnumConstantNameProperty settings with the new property;
…layEnumConstantProperty settings; implemented a central method for tooltip generation with max width and text wrap;
…l of preferences; general linting;
…ed up not using it yet;
…enceContainer.delete(String); switched FileUtil.deleteFile() to nio files; changed fail behaviour of time stamp methods;
…s and check of properties for preference restrictions to PreferenceUtil for SettingsContainer, FragmentationService, and ViewToolsManager;
To do: Write changelog and then merge. |
…te about JavaFx info message in log file when reducing the rows per page setting;
To do: separate names and display names of fragmenters. |
…ve integer filter including zero;
… double filter including zero;
…lay name for use in GUI; internal name is now mostly used for persistence;
Quality Gate passedIssues Measures |
Complete changelog of this branch / pull request:
|
No description provided.