search

symfony-cmf / seo-bundle

A SEO Solution for duplicate contents, page titles, etc.
https://cmf.symfony.com
47 stars 27 forks source link

Use Sonata defaults in form #143

Open wouterj opened 10 years ago

wouterj commented 10 years ago

The form in the admin panel should not only show the values of the SeoMetadata, but also the defaults of the Sonata page service if no value was found for it in SeoMetadata

dbu commented 10 years ago

would be nice, but must be done in a way that on save, we do not duplicate the defaults into the SeoMetadata.

ElectricMaxxx commented 10 years ago

:+1: @dbu It makes no sense to persist stuff, wich is set as default in cmf/sonata seo setting. In general We need a clear line of handling both kinds of default values. Which are used in which situation, that's an issue i have with the current version. But i would be a solution to just "show" the default values or show the result with the current setting - based on that clear line We need.

ElectricMaxxx commented 10 years ago

Btw i won't see this thing as a "bug". It's a feature for the admin form extension support. :-)

ElectricMaxxx commented 10 years ago

An other question would be: How do you want to display the existing values for all ExtraProperties. Would never be possible to show them while adding a new one. Whith sonatas "show-action" we could add a "view", means a list, of the current state after save. Maybe with a contrast between default and content's values.

dbu commented 10 years ago

maybe we should solve that by showing a hint with the base value that is going to be used if field is left empty?