Problem/Motivation

There is no @param and @return documentation for couple functions in the module.

Proposed resolution

Add the @param and @return

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 2616156-1.patch2.29 KBrakesh.gectcr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rakesh.gectcr created an issue. See original summary.

rakesh.gectcr’s picture

rakesh.gectcr
he/him
Primary language Malayalam
Location Manchester
CreditAttribution: rakesh.gectcr as a volunteer and at Valuebound commented
Status: Active » Needs review
FileSize
2616156-1.patch2.29 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2616156-1.patch, failed testing.

rakesh.gectcr’s picture

rakesh.gectcr
he/him
Primary language Malayalam
Location Manchester
CreditAttribution: rakesh.gectcr as a volunteer and at Valuebound commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: 2616156-1.patch, failed testing.

  • frjo committed a1b1d10 on 8.x-1.x authored by rakesh.gectcr 
    Issue #2616156 by rakesh.gectcr: Add @return type and @param values in...
frjo’s picture

frjo CreditAttribution: frjo commented

Committed, thanks for helping to improve the Colorbox module!

rakesh.gectcr’s picture

rakesh.gectcr
he/him
Primary language Malayalam
Location Manchester
CreditAttribution: rakesh.gectcr as a volunteer and at Valuebound commented

@frjo

Do we need to close this issue?

frjo’s picture

frjo CreditAttribution: frjo commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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