Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Develop #16

Merged
merged 18 commits into from
Feb 24, 2023
Merged

Develop #16

merged 18 commits into from
Feb 24, 2023

Conversation

aljoshakoecher
Copy link
Member

Introduces two fixes:

  1. Now both literals and IRIs are properly handled (see baa274b followed by 75e9614)
  2. Fixes a bug that incorrectly handled container paths with multiple attribute filters, e.g.
    /something[@name =..]/anotherThing[@name=123]. This fixes Fix problem with multiple quotation marks used in the container #15

@aljoshakoecher
Removed unneeded code
37afd47
@aljoshakoecher
Added data to default model so that graph doesn't have to be used in …
75534de 
…query
@aljoshakoecher
Now properly getting literal as string
baa274b
@aljoshakoecher
Added initial JSON mapping implementation
0ac676b
@aljoshakoecher
Added JSON tests along with test resources
e099c8c
@aljoshakoecher
Added dependency for JSON assertions
8340da7
@aljoshakoecher
Fixed wrong literal typing and now clearing results
93bb53a
@aljoshakoecher
Return to proper integer values after typing issues was fixed
c116ecb
@aljoshakoecher
Now using Jackson json and mapping providers as Gson didn't properly …
1ac6563 
…handle integers
@aljoshakoecher
Now using two separate tests to test a result that builds up on anoth…
61d4303 
…er one
@aljoshakoecher
Code cleanup
f699b0c
@aljoshakoecher
Renamed test folders (persons -> XML, pokemon -> JSON)
cfd0fc9
@aljoshakoecher
Added a method to properly get values for both Resources and Literals
75e9614
@aljoshakoecher
Swapped actual and expected as they were in the wrong order
de6bd65
@aljoshakoecher
Fixed RegEx that lead to multiple attribute filters causing problems
59dcc50
@aljoshakoecher
Added a test that checks multiple attribute filters
d82c866
@aljoshakoecher
Edited test folder paths
699c5ed
@aljoshakoecher
Renamed DataInterchangeFormat to DataExchangeFormat
166b3c0
@aljoshakoecher aljoshakoecher added the bug Something isn't working label Feb 24, 2023
@aljoshakoecher aljoshakoecher merged commit af22fdb into main Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix problem with multiple quotation marks used in the container
1 participant
@aljoshakoecher