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

[TASK] Sanitise Fluid integration #70

Conversation

NamelessCoder
Copy link
Contributor

Improves the Fluid integration in the following ways:

  • Adds an extension key context variable to AbstractWidget
    which allows a third-party widget to override the use
    of "dashboard" as extension key without having to override
    the initializeView method.
  • Uses this extension key to prefill the TemplatePaths used
    by the created view.
  • Renames the template folders to observe the usual MVC
    pattern used by Fluid in TYPO3 CMS and not require
    non-standard template root paths.
  • Renames the Layout to not require non-standard layout
    root paths.

Improves the Fluid integration in the following ways:

* Adds an extension key context variable to AbstractWidget
  which allows a third-party widget to override the use
  of "dashboard" as extension key without having to override
  the initializeView method.
* Uses this extension key to prefill the TemplatePaths used
  by the created view.
* Renames the template folders to observe the usual MVC
  pattern used by Fluid in TYPO3 CMS and not require
  non-standard template root paths.
* Renames the Layout to not require non-standard layout
  root paths.
@NamelessCoder
Copy link
Contributor Author

Requires rebase and adaptation of one additional template file if #69 is merged first.

@haassie haassie merged commit 65ae602 into TYPO3-Initiatives:master Oct 21, 2019
NamelessCoder added a commit to NamelessCoder/dashboard that referenced this pull request Oct 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants