Closed (works as designed)
Project:
Views (for Drupal 7)
Version:
7.x-3.x-dev
Component:
User interface
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Feb 2011 at 00:32 UTC
Updated:
1 Aug 2017 at 14:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dawehnerPlease search also on closed issues. This was reported already too many times.
Comment #2
danielnolde commentedSo what's the solution, Daniel ?!? What's the reference issue ?
Comment #3
danielnolde commentedokay, the solution is (in my case and probably many others) to deactivate WYSIWYG.module.
ARRRRGHH. Damn. That's no solution at all. A real solution would be:
for drupal developers to
1) code a workaround to disable wysiwyg for the field, e.g. via form_alter with $form[ELEMENT]['#wysiwyg'] = FALSE;
for views (co)maintainers to
2) give a one-word hint like "wysiwyg.moule" even if their eyes are sore with duplicate issues.
3) integrate a warning ("this could be broken because of wysiwyg.module - just in case you are among the 50% of users who dare to use views _and_ wysiwyg api")
for site builders to
4) spend the weekend crying
I'll give solution 1) a shot.
Daniel (dereine), perhaps you try 3) to lessen the sore in your issue-queue-manager-eyes in the future ?
(set to "active" so maintainer can see this ;)
Comment #4
merlinofchaos commentedIn an ideal world, wysiwyg would fix the problem so Views would quit getting blamed for it.
Comment #5
dawehnerhttp://drupal.org/project/issues/views?text=header&status=All&priorities...
Comment #6
dawehnerHere is a patch for solution 2)
Should such issues always have one open issue as placeholder?
Comment #7
danielnolde commentedwow, dereine is fast.
yes, wysiwyg is to "blame", but views could do something about it (and lessen then support load for views maintainers).
imho, one always-open-issue for this topic should do help people to find the cause of the problem quicker.
unfortunately, dereine's patch doesn't seem to work, perhaps the #wysiwyg property seems to be outdated: http://drupal.org/node/285200#comment-3892916
attached is a patch with a (manually) added warning that proposes three ways towyrds a solution right in the interface (if wysiwyg module is present).
Comment #8
danielnolde commentedanother quick fix could be for views to default to the plain text format (if it's available). what do you think? This is unobstrusive for the rest of the site (rather than suggesting to the site builder to change to text format order for the whole site).
Comment #9
danielnolde commentedattached is a edited patch that uses filter_fallback_format() to set the default format for the 'global text' (and still contains #wysiwyg and the warning). Not perfect, since even the fallback filter _could_ be configured to use wysiwyg api ... but better in any case. what do you think?
Comment #10
dawehnerChanging the default value might cause a problem.
If you have a certain value defined in option_definition and you don't change it in the form, the value is not stored in the export.
Once you change the default value the new value is loaded and boom you get a different output. This might be an edge-case.
I guess many people use the default format as format.
This is an example output. Some text was entered, but the format is not stored, additional.
Mh in general the value in option_definition should be plain text, but another #default_value should be set in the form itself, because
the filter_default_format() might change from installation to installation but your header should be the same in both.
So we should perhaps force to store the format always.
Comment #11
danielnolde commentedouuh, didn't think of that edge-case, but it's an important one. Forced storage might be a good solution (also because then chaning your default format won't override all your views in all your features, but that's another topic!).
Anyway, here's a quick fix solution anyone with a custom module can implement, that forces the fallback format (if not "php_code" is already used). Not perfect, since very forceful and bully-ish, but works in many cases (at least mine ;)
Comment #12
cindyr commentedThanks! Works great for me.
Comment #13
dawehnerAfter some thoughts about it. This is not only an edge case.
We have to support previous values.
So what about this patch?
Comment #14
adavis.co.uk commentedI've got this error too and am not using wysiwyg. I am using CKEditor but disabled the module to no effect.
Comment #15
anavarreDefinitely a Wysiwyg problem. When you use Firebug or Webkit inspector (Chrome/Safari), select the textarea tag, choose to dynamically delete "visibility:hidden" and Views textarea will work fine again. Deactivate Wysiwyg and the problem is gone.
Comment #16
bryancasler commentedTried the patch in #13, no success :(
Comment #17
ytsurk#11 works wonderfully
Comment #18
bryancasler commentedThanks #17!
Comment #19
dawehnerThe current patch doesn't really solve the problem, it just fixes some edge cases.
No idea how we should use filter_fallback_format without breaking existing views headers.
Comment #20
dawehnerTo be enoughst this still needs work.
There is no patch to solve the real problem without causing major problems with existing views.
Comment #21
dawehnerCommited a #wysiwyg => FALSE part.
Comment #22
dawehnerSo in general this could be marked as fixed from my perspective.
Comment #23
molave commentedNot sure if my problem is related or not, but I too am missing a text area in Views. In my case, however, it's the text area of the Global Custom Text field.
Right after creating a new View, it's easy to add a Global Custom Text field, and put in it some static text, and also place in it a token from the Content Title field.
After adding new Fields to the View, however, and upon returning to Global Custom Text to add more tokens, the text area is no longer there, so the "assembly" can no longer be modified. All other options (Style Settings, No Results Behavior, etc) are still available and editable. But no text area.
Strangeness, ja?
Running D7. As of April 2011, also installed are the latest versions of CTools, Token, Views, Backup and Migrate, Advanced Help, Administration Menu, Taxonomy CSV, JobScheduler, Feeds, References, Field Group, Webform, Features, Context, Quicktabs, Panels, PathAuto.
No WYSIWYG editor of any flavor installed.
As always, any insights appreciated.
Comment #24
dawehnerSee #1131424: Global: Custom Text cannot be configured for an issue about global custom field.
Comment #25
molave commented@dereine Thanks much for pointing me in the right direction. Will subscribe to that issue, then.
Comment #26
dawehnerwyswiyg false is implemented and the format is now defaulted to NULL.
Let's mark this issue as fixed
Comment #27
bryancasler commentedwoot
Comment #29
ionmedia commentedplease, tell me, is wysywig editor must work in views header ?
Comment #30
dawehnerYou ask whether the wysiwyg editor should be shown in views?
Comment #31
mustanggb commented