Currently you simply can't configure fields when you have some either a style or a row plugin which doesn't allow that.

For example you can't use fields to groupby content: type when you use content as row plugin.
Let's introduce a checkbox that adds the feature.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
FileSize
4.43 KB
1.67 KB

Here is a version that works for me. Suprisingly views actually already allows that, it's just some kind of configuration issue :)

tim.plunkett’s picture

Status: Needs review » Needs work
+++ b/plugins/views_plugin_style.incundefined
@@ -88,11 +88,15 @@ class views_plugin_style extends views_plugin {
+    return $row_uses_fields
+      ||!empty($this->definition['uses fields'])
+      || !empty($this->options['uses_fields']);

This could probably be one line, I think its more confusing this way. Also, there is a missing space between ||!empty

+++ b/plugins/views_plugin_style.incundefined
@@ -166,6 +170,7 @@ class views_plugin_style extends views_plugin {
+    $options['uses_fields'] = array('default' => FALSE);

Doesn't this need a 'bool' => TRUE for proper exporting? Though I notice from the patch context that others are missing it, perhaps that just needs a cleanup?

dawehner’s picture

Status: Needs work » Needs review
FileSize
1.76 KB

You are right, this is definitive harder to read then in one line.

Here is a new patch.

Regarding the bool issue, i'm working on a global cleanup patch.

tim.plunkett’s picture

FileSize
2.31 KB

I think the "force fields" checkbox should only be shown when valid. It will just be confusing if the style or row plugin use fields.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Oh that's a good idea. Now the patch looks RTBC

dawehner’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to 7.x-3.x, maybe it's helpful for 6.x-3.x as well.

spovlot’s picture

When will this fix be released?

dawehner’s picture

It is easy, somehow will write a patch, this patch will be committed and then everyone will be able to profit from it.

dman’s picture

I'm interested in this.
I'll have to test if it needs a re-roll?

spovlot’s picture

So, this fix has been modified in 7.x-3.x-dev, when will it be released as 7.x-3.4? It seems that the patch has already been written and committed to the codeline. Just trying to understand the process.

dman’s picture

There's no commit link, but based on the dates, with 7.x-3.3 = 2012-Feb-22 and no releases since then ... I guess yeah - we should be waiting on the next version number and it should be there.
My misunderstanding (re the re-roll) above was just confusion by me - I thought there had been a views release more recently than Feb.

spovlot’s picture

An alternate to using the "Content" row plugin is to use fields with the "Content: Rendered Node". Within the Rendered Node field settings, you can choose Display = "Show complete entity" and then choose a View Mode. The output provides the similar results to using the Content format. (Thanks to pixelwhip from http://drupal.org/node/1619340)

MustangGB’s picture

Issue summary: View changes
Status: Patch (to be ported) » Closed (won't fix)