-
Notifications
You must be signed in to change notification settings - Fork 751
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
Legifrance: fix error on import & parse more fields #2995
base: master
Are you sure you want to change the base?
Conversation
I'm getting a "no header" error on all linter executions, regardless of the file. |
IIRC no header warnings are typically warnings due to Windows line breaks. The linter only works with LF. As you can tell from the automated lint, you definitely want to run this through |
No, those latest changes don't make any sense -- we don't want all those empty lines in the code. Are you really not able to get lint --fix to work? Doing this manually is a complete waste of time. |
No, I tried switching to LF EOL, but it didn't fix the problem with the linter. (Sorry for not answering to your earlier message, I thought I should I should try solutions before coming back to you) |
OK, let me run this through the linter once here |
OK, I think that should pass the CI. I haven't looked at the code changes proper at all, though, so those still require review |
Thank you for the help Is there a file reference you would recommend for comments and code style? |
to revert? (done by VSC's autoformat)
– Auto-update of tags for each tests – Pinpoint missing items: –– attachments (website's fault), –– libraryCatalog (unused by case and statute), –– accessDate (handled by Zotero?)
Update XPATH Update regex Reconstruct nameOfAct to fit previous naming Update and add related tests
The origdate Xpath depends on the code; requires more study before implementation
Hello,
There was an error while parsing statutes from Légifrance (e.g., this page or this one)
It was also mentioned on the forum: https://forums.zotero.org/discussion/97635/probleme-denregistrement-sur-le-site-legifrance (in French)
It was due to a layout(?) change, and the title was no longer available at the previous XPath.
I changed the Xpath and added a few modifications, like adding missing
var
declarations.I'm not sure I will implement all changes I intended; it might be better to push the bug fixes earlier (and uni is more urgent).
I'm mostly making the PR draft right now to get some feedback, the linter output (doesn't work on my machine), and let you push as soon as you wish to do so.
I would be glad to have some feedback, as I'm fairly new to JS.
List of changes
Bug fixes
Translation failed: TypeError: title is null
errors on statutesFeature changes
Code/lint changes
processDocuments
Test changes