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

FixExecutionException: Expected Array or Hash, got String #1368

Closed
dr0i opened this issue Aug 8, 2022 · 1 comment · Fixed by #1369
Closed

FixExecutionException: Expected Array or Hash, got String #1368

dr0i opened this issue Aug 8, 2022 · 1 comment · Fixed by #1369
Assignees
@dr0i
Labels
ALMA bug
Milestone
ALMA-FIX

Comments

@dr0i
Copy link
Member

dr0i commented Aug 8, 2022

Following up #1341 following error for resource 991000688209706449 occurs:

[WARNING] 
org.metafacture.metafix.FixExecutionException: Error while executing Fix expression (at file:/home/sol/git/lobid-resources-alma-fix/src/main/resources/alma/fix/describedBy.fix, line 32): replace_all("describedBy.dateCreated|dateModified","-","")
IllegalStateException: Expected Array or Hash, got String
@dr0i dr0i added this to the ALMA-FIX milestone Aug 8, 2022
@dr0i dr0i added the bug label Aug 8, 2022
TobiasNx added a commit that referenced this issue Aug 9, 2022
@TobiasNx
Add testfile 991000688209706449 #1368
f2155ba
TobiasNx added a commit that referenced this issue Aug 9, 2022
@TobiasNx
Fix error #1368
70999b9 
It seems that alterations in path for `replace_all` automatically expects multiple matches in a record.
See: metafacture/metafacture-fix#249

Therefore I splitted the `replace_all` functions and created a function for each alteration.
@TobiasNx TobiasNx assigned dr0i and unassigned TobiasNx Aug 9, 2022
@TobiasNx TobiasNx added the ALMA label Aug 9, 2022
@dr0i dr0i mentioned this issue Aug 9, 2022
Merged
@dr0i
Copy link
Member Author

dr0i commented Aug 9, 2022

Done, cloing.

@dr0i dr0i closed this as completed Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dr0i dr0i

Labels
ALMA bug
Projects
None yet
Milestone
ALMA-FIX
Development

Successfully merging a pull request may close this issue.

1368 error fix hbz/lobid-resources
2 participants
@dr0i @TobiasNx