Latest Pareview report shows some doc formatting issues in ContactsService.

See the attached file.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

john_a created an issue. See original summary.

john_a’s picture

Status: Active » Needs review
FileSize
15.41 KB

Patch created, please review.

There are a few remaining pareview issues, but I'm ignoring these. The lines blow are short descriptions, which are lifted straight from online Salesforce documentation. They do not have paragraphs. So for completeness and readability, I'm ignoring the warnings here

----------------------------------------------------------------------
FOUND 0 ERRORS AND 13 WARNINGS AFFECTING 13 LINES
----------------------------------------------------------------------
23 | WARNING | Line exceeds 80 characters; contains 91 characters
36 | WARNING | Line exceeds 80 characters; contains 92 characters
52 | WARNING | Line exceeds 80 characters; contains 87 characters
68 | WARNING | Line exceeds 80 characters; contains 84 characters
100 | WARNING | Line exceeds 80 characters; contains 113 characters
148 | WARNING | Line exceeds 80 characters; contains 116 characters
154 | WARNING | Line exceeds 80 characters; contains 137 characters
169 | WARNING | Line exceeds 80 characters; contains 116 characters
207 | WARNING | Line exceeds 80 characters; contains 112 characters
224 | WARNING | Line exceeds 80 characters; contains 84 characters
240 | WARNING | Line exceeds 80 characters; contains 94 characters
256 | WARNING | Line exceeds 80 characters; contains 113 characters
299 | WARNING | Line exceeds 80 characters; contains 142 characters
---------------------------------------------------------------------

john_a’s picture

Created new patch one of the remaining max 80 chars issues could be resolved. The remaining are all SF short descriptions that should not be shortened, for clarity.

john_a’s picture

Status: Needs review » Fixed

patch reviewed, updated and applied by self

john_a’s picture

Status: Fixed » Closed (fixed)