Move edac_insert_rule_data to an inserter class

pattonwebz commented 6 months ago

Moves the edac_insert_rule_data function to a new Insert_Rule_Data class with a single method named insert that takes the same passed params.

I added type hints on the parameters (should post actually be a \WP_Post type rather than an object?).
Added an early bail if one of the required params aren't passed (typehints technically prevent this happening but having it makes it more clear what is going on).
Started adding tests (which still needs more work to cover more of the method).

I have some thoughts about potential refactor of this function so that it can have return values that are likely to give more certainty about what has happened. My initial thought was bool|int and never void. With true being success, false being fail and the int meaning it was a duplicate but would also evaluate to a truth value for checking that might be added later after it's called.

Closes: #464

SteveJonesDev commented 6 months ago

@pattonwebz, are you still working on this PR since it's in draft mode?

pattonwebz commented 6 months ago

@pattonwebz, are you still working on this PR since it's in draft mode?

I still need to work on the tests here for better coverage. If you want to review the actual code changes, that'd be great.

I plan to revisit the tests here tomorrow and the other issues assigned to me.

equalizedigital / accessibility-checker

Move edac_insert_rule_data to an inserter class #513