-
Notifications
You must be signed in to change notification settings - Fork 2
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
Namsonx/task/stabi branch #369
Merged
Merged
Changes from 24 commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
930824d
Updated self test and code snippet generator
HolQue cc544e2
Merge pull request #362 from test-fullautomation/HolQue/task/selftest…
namsonx d9b5acf
Code snippet generator: "blocked substitutions" snippets automated
HolQue f4c9eba
Merge pull request #368 from test-fullautomation/HolQue/task/selftest…
test-fullautomation a086d5a
Enhancement - Implemented dynamic path of imported file
namsonx 8f90062
Update history and version for 0.8.2
namsonx 495a655
Release info maintenance
HolQue 9ac0349
Merge pull request #375 from test-fullautomation/HolQue/task/selftest…
namsonx 87fc582
Updated dynamic paths for imported JSON files feature
namsonx 3eced23
Added some JSONP self test files (for tests of dynamic import)
HolQue d656227
Merge pull request #376 from test-fullautomation/HolQue/task/selftest…
test-fullautomation 4200b1d
Added further JSONP self test files (for tests of dynamic import)
HolQue da95ec8
Merge pull request #378 from test-fullautomation/HolQue/task/selftest…
test-fullautomation 9d01acd
Added further JSONP self test files (for tests of dynamic import, imp…
HolQue 12da242
Merge pull request #379 from test-fullautomation/HolQue/task/selftest…
test-fullautomation 728b8d8
Added FILE_IMPORTS self test cases
HolQue cd344e0
Merge pull request #384 from test-fullautomation/HolQue/task/selftest…
test-fullautomation cb510cd
Ticket 371 - Enhancement Dynamic import feature
namsonx 13418c5
Updated self test
HolQue 5be1b19
Merge pull request #386 from test-fullautomation/HolQue/task/selftest…
test-fullautomation cb3e03c
Enhance dynamic import feature based on test case 1104
namsonx 65b94a8
Enhance and stabilize dynamic import feature
namsonx 4dc0fd6
Enhance and stabilize dynamic import feature - continue
namsonx c9b9d40
Ticket 370 - Missing parameter in dynamic import
namsonx 2099ab7
Self test adapted to latest changes in JsonPreprocessor code
HolQue 0eab640
Merge pull request #388 from test-fullautomation/HolQue/task/selftest…
test-fullautomation ea28e95
Ticket 383 - Update error message for dynamic import feature
namsonx 855d7cb
Ticket 383 - Tiny update due to typo.
namsonx 2fc7b76
Ticket 381 and Enhancement - Invalid syntax freezes the JsonPreprocessor
namsonx cfc62de
Ticket 380 - Cyclic import with dynamic paths not detected
namsonx File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
Binary file not shown.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -138,28 +138,42 @@ | |
|
||
* Added a naming convention check for key names within JSONP content processed by the **JsonPreprocessor** | ||
|
||
Key names have to start with a character, digit, or underscore and must not contain these special characters ``!#$%^&()=[]{}|;',?`~`` | ||
Naming convention: | ||
|
||
* Key names can only consist of letters, digits and the following special characters: ``_ + - * / \\`` (backslashes are allowed but must be masked). | ||
* Key names must start with a letter, a digit or an underscore. | ||
* Key names must not be empty strings. But leading and trailing blanks will be removed (and therefore do not cause errors). | ||
|
||
**Example:** | ||
|
||
Valid key names are: ``\"abcParam\"``, ``\"01_Param\"``, ``\"__param+1\"``, ``\"param-1\"``, ``\"[email protected]\"``, ... | ||
|
||
Invalid key names are: ``\"+param01\"``, ``\"param$01\"``, ``\"abc#Param\"``, ... | ||
|
||
* Checked absolute path when overwriting parameter | ||
* Reworked handling of parameter scope | ||
|
||
When overwriting a parameter, the absolute path of the parameter must be provided | ||
To change the value of an existing parameter, an absolute path must be used always. | ||
|
||
**Example:** | ||
|
||
|``\u007b`` | ||
| ``\"params\" : \u007b\"001\" : \u007b\"002\" : \u007b`` | ||
| ``\"param\" : 1,`` | ||
| ``$\u007bparams.001.002.param\u007d : 2`` | ||
| ``\u007d`` | ||
| ``\u007d`` | ||
| ``\u007d`` | ||
|``\u007d`` | ||
| ``{`` | ||
| ``\u00a0\u00a0\u00a0\"params\"\u00a0:\u00a0{\"001\"\u00a0:\u00a0{\"002\"\u00a0:\u00a0{`` | ||
| ``\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\"param\"\u00a0:\u00a01,`` | ||
| ``\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0${params.001.002.param}\u00a0:\u00a02`` | ||
| ``\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0}`` | ||
| ``\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0}`` | ||
| ``\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0\u00a0}`` | ||
| ``}`` | ||
|
||
* Enabled dynamic paths for imported JSON files (based on dollar operator expressions) | ||
|
||
**Example:** | ||
|
||
| ``{`` | ||
| ``\u00a0\u00a0\u00a0\"root_folder\" : \"imports\",`` | ||
| ``\u00a0\u00a0\u00a0\"json_file\"\u00a0\u00a0\u00a0: \"configuration.jsonp\",`` | ||
| ``\u00a0\u00a0\u00a0\"[import]\"\u00a0\u00a0\u00a0\u00a0: \"./${root_folder}/${json_file}\" `` | ||
| ``}`` | ||
" | ||
] | ||
} | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Hi Son,
please rephrase to:
Enabled dynamic paths for imported JSON files (based on dollar operator expressions)