A previos patch introduces notices when no group exists:

Notice: Undefined offset: 0 en include() (line 15 sites/all/modules/contrib/views_datasource/views/theme/views-views-json-style-simple.tpl.php).
Notice: Undefined index: en include() (line 18 de sites/all/modules/contrib/views_datasource/views/theme/views-views-json-style-simple.tpl.php).
Notice: Trying to get property of non-object en include() (línea 18 de sites/all/modules/contrib/views_datasource/views/theme/views-views-json-style-simple.tpl.php).
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

GeduR created an issue. See original summary.

GeduR’s picture

Status: Active » Needs work
FileSize
1.69 KB

Please review

GeduR’s picture

Issue summary: View changes
GeduR’s picture

Status: Needs work » Needs review
jacob.embree’s picture

Status: Needs review » Reviewed & tested by the community

Nice patch.

Marceldeb’s picture

Very nice patch indeed!

hgoto’s picture

Thank for the patch. The patch #2 works well. +1 for RTBC.

hgoto’s picture

"previos patch" in the summary points the following patch.

#1373280-5: Grouping for nodes 7.x

nithinkolekar’s picture

RTBC
patch tested manually..

rjbrown99’s picture

#2 applied fine for me and worked to resolve the problem. It's already RTBC but wanted to chime in with one more positive result.

  • ZeiP committed b579843 on 7.x-1.x authored by GeduR
    Issue #2830934 by GeduR: Remove notices on group style simple
    
ZeiP’s picture

Status: Reviewed & tested by the community » Fixed

Looks good, thanks everyone! Committed the patch to 7.x-1.x.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.