For people trying to get familiar with the code, it would really help documenting the helper functions according to the drupal coding standards.

Members fund testing for the Drupal project. Drupal Association Learn more


marcoscano created an issue. See original summary.

marcoscano’s picture

Assigned: marcoscano » Unassigned
Status: Active » Needs review
6.91 KB

This is not exhaustive, I have checked only the .module file.
(solved also some other minor CS nitpicks along the way)

Status: Needs review » Needs work

The last submitted patch, 2: improve-docblocks-2783183-2.patch, failed testing.

naveenvalecha’s picture

Issue tags: +Needs reroll

patch no longer applies

ajalan065’s picture

Assigned: Unassigned » ajalan065

I will post the new patch with improved documentation.

eugene.ilyin’s picture

This comment looks strange for me

/** @var FieldDefinitionInterface $field */

ajalan065’s picture

Assigned: ajalan065 » naveenvalecha
Status: Needs work » Needs review
Issue tags: -Needs reroll
3.65 KB

Here is the rerolled patch for updated documentation.

naveenvalecha’s picture

Assigned: naveenvalecha » Unassigned
Status: Needs review » Reviewed & tested by the community

Looks good we are close to the coding standards fixes :)

naveenvalecha’s picture

As mentioned you earlier in previous issue about the patch name.Consider it unless you are bypassing the drupal ci on every patch. it would be hard for us to reupload it, however its fine for me b/c I test patches on my local dev box but consider the standard approach that we are using.

naveenvalecha’s picture

Category: Feature request » Task
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x

Status: Fixed » Closed (fixed)

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