Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REF] APIv4 - Rename trait 'SoftDelete' to 'SoftDeleteEntity' #22662

Merged
merged 1 commit into from
Feb 2, 2022

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jan 29, 2022

Overview

Simple file rename for consistency with other traits

Before

APIv4 has some "ability" traits, named:

  • ManagedEntity
  • SortableEntity
  • ReadOnlyEntity
  • SoftDelete

After

  • SoftDeleteEntity

@civibot civibot bot added the master label Jan 29, 2022
@civibot
Copy link

civibot bot commented Jan 29, 2022

(Standard links)

@colemanw
Copy link
Member Author

@braders - this would be a good one for you to review

@braders
Copy link
Contributor

braders commented Jan 29, 2022

@colemanw Honestly I'm not that familiar with the API4 code, but this looks sensible enough.

The only concern that springs to mind is that extensions could be using this trait - you'll know better than me if that's a legitimate concern or not.

@colemanw colemanw added the merge ready PR will be merged after a few days if there are no objections label Jan 29, 2022
@colemanw
Copy link
Member Author

In general that's a smart thing to consider @braders. In this particular case I would say very unlikely because the trait is new, internal, and not documented anywhere. However I vaguely recall @monishdeb working on some extension code where I suggested this trait might be useful to him, so I'll just add merge-ready here and give him a chance to double-check that.

@demeritcowboy
Copy link
Contributor

Github still thinks jenkins is queued for this but it appears to have passed tests and is pretty straightforward.

@demeritcowboy demeritcowboy merged commit 1d315b3 into civicrm:master Feb 2, 2022
@colemanw colemanw deleted the softDeleteEntity branch February 2, 2022 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master merge ready PR will be merged after a few days if there are no objections
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants