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

Implemented is-published function for route-provider #7

Merged
merged 2 commits into from
Jul 27, 2016

Conversation

wachterjohannes
Copy link
Member

@wachterjohannes wachterjohannes commented Jul 25, 2016

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Fixed tickets fixed #4
Related issues/PRs sulu/sulu#2669
License MIT
Documentation PR none

What's in this PR?

This PR fixes the defaults-provider for unpublished articles.

Why?

If an article was never released there was an error.

@wachterjohannes wachterjohannes added the bug Error or unexpected behavior of already existing functionality label Jul 25, 2016
@wachterjohannes wachterjohannes force-pushed the bugfix/404-for-unpublished branch from 8a5969d to b223385 Compare July 25, 2016 13:22
->useAttributeAsKey('name')
->prototype('array')
->children()
->arrayNode('translations')
Copy link
Member

Choose a reason for hiding this comment

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

Use translations instead of config.

@wachterjohannes wachterjohannes force-pushed the bugfix/404-for-unpublished branch from b223385 to d02e877 Compare July 27, 2016 10:08
@wachterjohannes wachterjohannes force-pushed the bugfix/404-for-unpublished branch from d02e877 to 8c97a70 Compare July 27, 2016 10:14
@chirimoya chirimoya merged commit f035aa7 into develop Jul 27, 2016
@chirimoya chirimoya deleted the bugfix/404-for-unpublished branch July 27, 2016 10:17
trickreich added a commit that referenced this pull request Jul 31, 2018
* Webspace settings frontend

* Fix StyleCI

* Fixed bug

* PHPCR Implementation (#4)

* Implementation finished

* Add tests

* Add ES indexing

* Info just gray

* Check for ES in test

* Add install notes

* Better doc

* Better doc

* Fix bug in addition webspaces preselect

* Fixed bug

* Dataprovider considers webspace settings (#5)

* Implementation finished

* Dataprovider considers webspace settings

* Code clean up

* Corrected property param

* Update ArticleDataProvider.php

* Resolve Sitemap correctly (#6)

* Add Canonical Tag (#7)

* Add Canonical Tag

* Code clean up

* Target webspace (#8)

* Add targetWebspace to ViewDocument

* Fixed test

* Search Subscriber (#10)

* ArticleSearchSubscriber

* Code clean up

* Fix tests

* Add "ignoreWebspaces" param to twig extension (#9)

* Add "ignoreWebspaces" param to twig extension

* Code clean up

* Finishing (#11)

* Add UPGRADE note

* Add NL translations

* Add FR translation

* Fix indention

* Fix StyleCI

* Page tree route (#12)

* Page tree route

* Disable webspace settings form

* Fix StyleCI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Error or unexpected behavior of already existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Type translation
2 participants