Closed (won't fix)
Project:
Nodewords: D6 Meta Tags
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
28 Oct 2010 at 20:17 UTC
Updated:
25 Sep 2020 at 14:57 UTC
Jump to comment: Most recent
Comments
Comment #1
dave reidIs this going to mess up the functionality of not wanting to provide specifically no meta tags for an item rather than the default meta tags?
Comment #2
katbailey commentedUgh - is that really how that works? In order to not have default tags added to a node you have to send empty nodewords content to the db? Or is it not the case that when you're creating a node, the default content for the various tags gets added as default values in the form and thus would get saved as non-empty nodewords unless deleted before the form was submitted? I'll admit I'm not entirely familiar with the multitude of ways this module can be used...
Comment #3
quicksketchThis is still a good issue to track. I'd say that our strategy should be as follows:
- If a default value is set and the default is unchanged when saving a metatag, nothing should be saved to the database.
- If a default value is not set and the field is left blank, nothing should be saved to the database.
- If a default value is set and the default is changed, the values should be saved to the database.
- If a default value is set and the default is left blank, the blank value should be saved to the database.
Or in a chart, here's what it would look like:
Comment #4
mcurry commentedsubscribing
Comment #5
avpadernoI see two problems with that.
How can the module know if nothing saved in the database means to use the default value or the empty value?
If the default values are changed, what should the module do? Should the module find any meta tag without a value in the database, and save the default value for them? If the answer to the last question is yes, how can the module distinguish between the case where the meta tag value has been left blank, and the case where the default value has not been changed?
Comment #6
damienmckennaI closed #1145010: Database performance and space efficiency as a duplicate of this.
Comment #7
damienmckennaTo properly handle this we'd almost need a checkbox to go with each tag to decide if the override should be used, then a bit of JS gravy to enable/disable the text field. Obviously that would need API changes and would make the module more difficult to use.
Comment #8
damienmckennaBumping this to v2.
Comment #9
damienmckennaMarking all v2 issues as postponed while v1 is finished off.
Comment #10
damienmckennaUnfortunately this module is no longer supported, so I'm closing this issue.