API page: https://api.drupal.org/api/drupal/core%21modules%21views%21views.theme.i...

The documentation for the $variables parameter shows the views row as being $variables['options']['row'] when it should be $variables['row'].

The example source code is correct.

Issue fork drupal-3119513

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

imclean created an issue. See original summary.

imclean’s picture

imclean commented

It looks like an HTML formatting issue. The opening and closing tags for the lists aren't in the right place. I suspect it should be something like this:

<h3>Parameters</h3>
<p>
  <strong>array $variables</strong>:
  An associative array containing:</p>
<ul>
  <li><strong>view</strong>: The view object.</li>
  <li><strong>options</strong>: An array of options. Each option contains:
    <ul>
      <li><strong>inline</strong>: An array that contains the fields that are to be
        displayed inline.</li>
      <li><strong>default_field_elements</strong>: If default field wrapper
        elements are to be provided.</li>
      <li><strong>hide_empty</strong>: Whether the field is to be hidden if empty.</li>
      <li><strong>element_default_classes</strong>: If the default classes are to be added.</li>
      <li><strong>separator</strong>: A string to be placed between inline fields to keep them
        visually distinct.</li>
    </ul>    
  <li><strong>row</strong>: An array containing information about the current row.</li>      
</ul>

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.7 was released on June 3, 2020 and is the final full bugfix release for the Drupal 8.8.x series. Drupal 8.8.x will not receive any further development aside from security fixes. Sites should prepare to update to Drupal 8.9.0 or Drupal 9.0.0 for ongoing support.

Bug reports should be targeted against the 8.9.x-dev branch from now on, and new development or disruptive changes should be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 8.9.x-dev » 9.2.x-dev

Drupal 8 is end-of-life as of November 17, 2021. There will not be further changes made to Drupal 8. Bugfixes are now made to the 9.3.x and higher branches only. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.2.x-dev » 9.3.x-dev

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.15 was released on June 1st, 2022 and is the final full bugfix release for the Drupal 9.3.x series. Drupal 9.3.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.4.x-dev branch from now on, and new development or disruptive changes should be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.9 was released on December 7, 2022 and is the final full bugfix release for the Drupal 9.4.x series. Drupal 9.4.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.5.x-dev branch from now on, and new development or disruptive changes should be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.5.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

quietone’s picture

quietone commented
Issue tags: +Novice

Prem Suthar made their first commit to this issue’s fork.

Prem Suthar opened merge request !8920

prem suthar’s picture

prem suthar commented
Status: Active » Needs review

Update the Changes of Parameter Documents. Please Review it.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Pretty straight forward just adding a space.

Also this issue was tagged novice less then a week ago, intended for new user. @Prem Suthar based on the post history that would not be you. Would recommend avoiding novice level issues in the future. May not have sent directly to you but know a similar message has been conveyed to other members of cmsMinds so please circulate that.

Now if a novice issue sits ideal for 3-4 weeks then that would be a different story, but not the case here.

nod_ closed merge request !8920

  • nod_ committed 202efea7 on 10.3.x 
    Issue #3119513 by Prem Suthar, imclean, smustgrave: Parameters doc for...

  • nod_ committed be2804a4 on 10.4.x 
    Issue #3119513 by Prem Suthar, imclean, smustgrave: Parameters doc for...

  • nod_ committed 696b56be on 11.0.x 
    Issue #3119513 by Prem Suthar, imclean, smustgrave: Parameters doc for...

  • nod_ committed fe4afc9a on 11.x 
    Issue #3119513 by Prem Suthar, imclean, smustgrave: Parameters doc for...
nod_’s picture

nod_
Primary language French
Location Lille
commented
Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed fe4afc9a13 to 11.x and 696b56bec1 to 11.0.x and be2804a4c2 to 10.4.x and 202efea7bd to 10.3.x. Thanks!

prem suthar’s picture

prem suthar commented

@smustgrave
I have checked, and the issue was created on March 12, 2020. Since then, no updates have been made, that's why I decided to address it. and Moving forward, I will keep this in mind and ensure that my team members avoid similar issues in the future.

Status: Fixed » Closed (fixed)

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