Deduplicate pathToVault.resolve("d").resolve("xx").resolve("yyyyyy...y")
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
- Issue type
- Refactor
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- java
- Domain
- operating-systems
Research direction
Search the project for the repeated pathToVault.resolve(...) expression and the dirHash calculation, then compare the places where both are used. The work is done when those call sites share a clean existing or injectable location rather than a new standalone utility class.
Written by the indexing model from the issue text.
Description
The expression pathToVault.resolve(Constants.DATA_DIR_NAME).resolve(dirHash.substring(0, 2)).resolve(dirHash.substring(2)) (or similar) is used in several places in this project. We should find a (clean!) way to deduplicate this (possibly including the dirHash calculation).
By "clean", I mean not just a new util class. Maybe we find some existing place, that can be injected (or is already present) in all required places.
- Dominant language
- Java
- Stars
- 110
- Forks
- 39
- Avg merge
- 1d 28m
- Merged PRs (30d)
- 4
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from cryptomator/cryptofs
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
cryptomator/cryptofs#372 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
cryptomator/cryptofs#246 · 6 comments ·
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
cryptomator/cryptofs#177 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
cryptomator/cryptofs#171 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
cryptomator/cryptofs#153 ·
All issues in cryptomator/cryptofs
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
wultra/powerauth-tests#980 ·
-
account bug communication lecture
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
line/centraldogma#1397 ·
Maintainers usually reply within 2 days
-
[Bug]: Fixed/recurring deposit accounts are treated as savings in ownership and listingsPossibly taken @Aana123 claimed this today. Openbug Triage
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
apache/fineract-consumer-facing#252 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 78/100
cryptomator/hub#515 ·