humanmade / humanmade/hm-content-import

Discussion: Abstraction/coupling of importers / and object insertion

Open
#11 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
PHP
Stars
30
Forks
7
Avg merge
6h 39m
Merged PRs (30d)
2

Description

Right now a custom importer extends a base importer ( and let me call that _a reader_ for now on for clarify ), and utilizes an _insert_ method within another base class for inserting content.

I think we need to reword the base importers ( files / csv / etc.. ) to be something like 'readers', and have another breed of classes for object handling like posts/users ( which already exists ), then custom importers ( probably the best name for these ) to couple a reader with an object, probably with a native method that reads a class variable which contains a mapping of attributes from reader input to an object output.
This will serve basic implementation pretty well, almost no logic to write. Comes to mind as L'express migration is that basic that it needs almost no additional logic but mapping existing fields to object attributes.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the base importers for files and CSV, the base class containing insert, and the existing posts/users and custom importer classes. The issue needs an agreed reader/object architecture, naming, and attribute-mapping approach before implementation can be considered done.

Written by the indexing model from the issue text.

Assessment

Tech stack
php, wordpress
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.