nextcloud / nextcloud/cookbook

Make outputing of XML errors optional

Open
#1,318 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

API Backend enhancement Frontend javascript php
Dominant language
HTML
Stars
641
Forks
113
Avg merge
21h 31m
Merged PRs (30d)
26

Description

Quite some issues are reporting issues with non-standard HTML files during import. Normally, this does not cause any real problems, the XML parser is possible to cope with some errors.

It would be a good idea to have an option per user to enable or disable the logging of these messages. That way, the bug reports are not too cluttered and not biased to the HTML parser.


Depends on #1067

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the import flow and the XML parser error logging described in the issue, then check the dependency in #1067 for related context. Done means users have a per-user option to enable or disable these messages during import, with the behavior covered by the relevant existing tests.

Written by the indexing model from the issue text.

Assessment

Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.