Consolidate functions for replacing path placeholder with path location
Nobody has claimed this yet.
- Dominant language
- MATLAB
- Stars
- 1
- Forks
- 2
- Avg merge
- 2h 18m
- Merged PRs (30d)
- 39
Description
The following functions seem to have some overlap:
in did.document/readjsonfilelocation:
https://github.com/VH-Lab/DID-matlab/blob/f4248688d49c68030ec2ebf68f4f92860969f281/code/%2Bdid/document.m#L728
in did.database/get_document_schema
https://github.com/VH-Lab/DID-matlab/blob/f4248688d49c68030ec2ebf68f4f92860969f281/code/%2Bdid/database.m#L1051
did.common.utility. replace_didpath
https://github.com/VH-Lab/DID-matlab/blob/f4248688d49c68030ec2ebf68f4f92860969f281/code/%2Bdid/%2Bcommon/%2Butility/replace_didpath.m#L1
Any other places?
Could consolidate this into a set of utility functions.
Contributor 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.
Research direction
Start by comparing did.document/readjsonfilelocation in code/+did/document.m, did.database/get_document_schema in code/+did/database.m, and did.common.utility.replace_didpath in code/+did/+common/+utility/replace_didpath.m. Search the repository for other path-placeholder replacement logic, then define the shared utility boundaries and verify that the affected document and database behavior remains consistent.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- matlab
- Domain
- database, tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 28/100