Skip to content

Added loaders and dumpers for nette database and Doctrine DBAL.#86

Closed
racinmat wants to merge 69 commits into
Kdyby:masterfrom
xcorecms:master
Closed

Added loaders and dumpers for nette database and Doctrine DBAL.#86
racinmat wants to merge 69 commits into
Kdyby:masterfrom
xcorecms:master

Conversation

@racinmat

Copy link
Copy Markdown

Added intergration of translations stored in database using doctrine DBAL and nette database.

Comment thread composer.json Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

driver: doctrine

nebo

driver: netteDb

nebo

driver: Trida\Ktera\Implementuje\Loader\Interface

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

asi bude lepsi pouzit Compiler::parseService, pak muze uzivatel lepe specifikovat argumenty, setup apod. to same u loaderu

@matej21

matej21 commented Sep 13, 2015

Copy link
Copy Markdown
Contributor

a oprav code style :) na spousta mistech je odsazeno mezerama, nekde jsou male true, false, null a na par mistech zustal nejaky debug vystup pres Debugger::log . a tam, kde nezustal, zustal v importech nepouziti Tracy\Debugger :D

@racinmat

Copy link
Copy Markdown
Author

Přidat index pro sloupec updated_at v překladové tabulce.
Použít cache na ukládání nejnovějšího updated_at, aby se nemusel při každém requestu pokládat dotaz.
Hotovo.

@fprochazka fprochazka mentioned this pull request Oct 17, 2015
@fprochazka

Copy link
Copy Markdown
Member

@racinmat thank you for your work, let's continue here #89

@fprochazka fprochazka closed this Oct 17, 2015
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.

7 participants