In ctools/views_content/plugins/content_types/views_panes.inc:

278   if ($allow['fields_override']) {
279     if ($conf['fields_override']) {
280       $fields = $view->get_items('field');
281       foreach ($conf['fields_override'] as $field => $display) {
282         $fields[$field]['exclude'] = empty($conf['fields_override'][$field]);
283       }
284       $view->display_handler->set_option('fields', $fields);
285     }
286   } 

You see on line 282, it bases the exclusion of the fields only on whether the config of the fields override is set or not. This can lead to a scenerio where a view is built, the fields override config is set to FALSE (= show the field). Later, the field will always show up if the field is excluded even in the views UI.

here is a patch. I feel there is more work to be done for the admin form. but this is a start.

CommentFileSizeAuthor
#2 ctools-874308-2.patch516 bytesjonskulski
#1 ctools-874308.patch4.21 KBjonskulski

Comments

jonskulski’s picture

Title: Fields excluded in the views object/UI will still show if the fields override conf is not correct » Fields excluded via fields override conf overrides exlusion set via views object/UI
StatusFileSize
new4.21 KB

Ignore this patch... wrong one.

jonskulski’s picture

StatusFileSize
new516 bytes

This is the patch I meant.

merlinofchaos’s picture

If I read your patch right, that means you can *never* display fields that were marked exclude in the Views UI.

This is not the behavior I'm looking for.

If you check that you are overriding which fields to show, then it should show exactly the fields checked, and no other field, right?

merlinofchaos’s picture

Also, please use cvs diff -up to generate patches.

merlinofchaos’s picture

Status: Needs review » Needs work

Moving this to needs work.

japerry’s picture

Issue summary: View changes
Status: Needs work » Closed (outdated)

Closing this issue as outdated as Drupal 6 ctools is not supported. If this issue is relevant for Drupal 7, feel free to re-open and mark for Drupal 7 (or 8)