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

[SeoBundle] Fixed issue that OG image can not be deleted. #434

Closed

Conversation

bureaublauwgeel
Copy link

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets 428

@roderik roderik changed the title Fixed issue that OG image can not be deleted. [SeoBundle] Fixed issue that OG image can not be deleted. May 26, 2015
@roderik
Copy link
Contributor

roderik commented May 26, 2015

@krispypen 👍

@@ -350,7 +350,7 @@ public function getOgDescription()
*
* @return Seo
*/
public function setOgImage(Media $ogImage)
Copy link
Contributor

Choose a reason for hiding this comment

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

Removing the type is not the fix for a null-exception. Can you change this to "Media $ogImage = null" ?

Copy link
Author

Choose a reason for hiding this comment

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

Should be changed.

woutervandamme and others added 9 commits May 26, 2015 16:07
added slugifier service to fixtures

add slugifier service to NodeBundle

updated slugifier tests

inject slugifier into utilitiesTwigExtension

twigExtension now works with the slugifier service

removed @Covers in slugifierTest

implemented slugifierInterface
…config-support

[AdminBundle] Added option to use a custom CKEditor config.js file
[GeneratorBundle] Refactor pagepart media validation
[AdminBundle] Added an option to change the backend site name to a logo
[AdminBundle,GeneratorBundle,NodeBundle,UtilitiesBundle] Change slugifier into service
[AdminBundle] Fix drag and drop sub entities
…l/KunstmaanBundlesCMS into feature/issue_428

* 'feature/issue_428' of https://github.com/bureaublauwgeel/KunstmaanBundlesCMS:
  Fixed issue that OG image can not be deleted.
wimvds added a commit to wimvds/KunstmaanBundlesCMS that referenced this pull request May 28, 2015
@wimvds
Copy link
Contributor

wimvds commented May 28, 2015

Replaced with #452

@wimvds wimvds closed this May 28, 2015
@bureaublauwgeel bureaublauwgeel deleted the feature/issue_428 branch June 2, 2015 08:19
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.

None yet

7 participants