Skip to content

[5.0][Events] Schemaorg event classes#41327

Merged
HLeithner merged 8 commits intojoomla:5.0-devfrom
heelc29:5.0/schemaorg/events
Aug 22, 2023
Merged

[5.0][Events] Schemaorg event classes#41327
HLeithner merged 8 commits intojoomla:5.0-devfrom
heelc29:5.0/schemaorg/events

Conversation

@heelc29
Copy link
Contributor

@heelc29 heelc29 commented Aug 6, 2023

Summary of Changes

Implementing Event classes for Schemaorg events.

Affected events:

  • onSchemaBeforeCompileHead
  • onSchemaPrepareData
  • onSchemaPrepareForm
  • onSchemaPrepareSave

Testing Instructions

  • Apply patch (and delete autoload_psr4.php)
  • Create schemas for articles and contacts in backend
  • Check output in frontend

Actual result BEFORE applying this Pull Request

works but support no concrete events

Expected result AFTER applying this Pull Request

works with legacy and concrete events

Link to documentations

Please select:

  • Documentation link for docs.joomla.org: Plugin/Events#Schemaorg
  • No documentation changes for manual.joomla.org needed

@HLeithner
Copy link
Member

@Fedik can you have a look at this please?

@Fedik
Copy link
Member

Fedik commented Aug 21, 2023

Looks good, by code review

@heelc29
Copy link
Contributor Author

heelc29 commented Aug 21, 2023

conflicts resolved

@HLeithner HLeithner merged commit b5a1c65 into joomla:5.0-dev Aug 22, 2023
@HLeithner
Copy link
Member

thanks

@heelc29 heelc29 deleted the 5.0/schemaorg/events branch September 3, 2023 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants