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

[DDST-455] BCELN24MIG-161: Feature/base fields #77

Merged
merged 7 commits into from
Aug 26, 2024

Conversation

adam-vessey
Copy link
Contributor

No description provided.

Soft-dependency on `base_field_override_ui`. When enabled, should allow
mapping of the CSL values via base fields.
@adam-vessey adam-vessey added the minor Added functionality that is backwards compatible. label Jul 31, 2024
src/Controller/IslandoraCitationsController.php Outdated Show resolved Hide resolved
islandora_citations.routing.yml Outdated Show resolved Hide resolved
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty much just:

  • renamed the old function away from the hook name
  • adjusted its contents to exit early when failing to load the "schema", where much of mess comes from here with reindentation
  • readded the old hook to call the new renamed function, as well as two new hooks which call the renamed function to similarly attach when both adding and editing base field overrides.
  • slightly simplified the form flow: Putting things directly into $form['third_party_settings'] avoids have to separately handling things on submission.

@rosiel
Copy link

rosiel commented Aug 5, 2024

With drupal/base_field_override_ui, i can map the title! Many thanks!

@JojoVes JojoVes changed the title BCELN24MIG-161: Feature/base fields [DDST-455] BCELN24MIG-161: Feature/base fields Aug 20, 2024
Comment on lines +118 to +121
new Link(
$this->t('Add'),
$add_destination(BaseFieldOverrideUI::getAddRouteInfo($config)),
) :
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An outdated comment is applicable to this section of code so linking for future reference #77 (comment)

And yes I can confirm this is currently not applicable to the list of fields being iterated on, but I see no harm keeping it here in case things get extended/altered to iterate over all base fields rather than just the ones already included in a given content type.

@JojoVes JojoVes marked this pull request as ready for review August 20, 2024 16:48
@chrismacdonaldw chrismacdonaldw merged commit 580d5ef into main Aug 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Added functionality that is backwards compatible.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants