FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[WIP] Zf3 by raykolbe · Pull Request #73 · raykolbe/DOMPDFModule · GitHub

[WIP] Zf3 - #73

Open
raykolbe wants to merge 8 commits into
masterfrom
zf3
Open

[WIP] Zf3#73
raykolbe wants to merge 8 commits into
masterfrom
zf3

Conversation

raykolbe commented Nov 25, 2017
edited
Loading

Copy link
Copy Markdown
Owner

Change Profile

Question Answer
New feature yes
Bug fix no
BC breaks yes
Passing tests yes

Description

⚠️ Work in progress: Upgrade to support ZF3.

Notable Breaking Changes

  • Namespace changed from DOMPDFModule to DompdfModule.
  • Service name change from DOMPDF to Dompdf.
  • No longer works with ZF2.
  • Moved source files from src/DOMPDFModule to src.
  • Moved Module.php to src/Module.php

Reason

Ensure continued support of ZF framework.

raykolbe added this to the 0.6.0 milestone Nov 25, 2017
raykolbe self-assigned this Nov 25, 2017
Comment thread config/module.config.php Outdated
'ViewPdfStrategy' => __NAMESPACE__ . '\Mvc\Service\ViewPdfStrategyFactory',
'Dompdf' => Service\DompdfFactory::class,
'ViewPdfRenderer' => Mvc\Service\ViewPdfRendererFactory::class,
'ViewPdfStrategy' => Mvc\Service\ViewPdfRendererFactory::class

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@raykolbe This should read Mvc\Service\ViewPdfStrategyFactory::class rather than RendererFactory::class.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Good catch.

svencan commented May 23, 2018
edited
Loading

Copy link
Copy Markdown

Any update on this? It would be nice if this was finally usable via composer as 0.6.0.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL