Download & Extend

Eliminate coder module warnings

Project:FeedAPI
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Patch is attached.

AttachmentSize
feedapi_coder_warnings.patch4.6 KB

Comments

#1

Status:needs review» fixed

Thank you, this is really awesome, i applied your patch and run through the coder module on other feedapi modules (aggregator, for example), so now all feedapi for 5.x is coder-safe :)

#2

Assigned to:Anonymous» mustafau
Status:fixed» needs work

It seems that coder module does not catch everything. I found some other errors. Patch is attached.

AttachmentSize
feedapi-more_coder_format.patch 4.92 KB

#3

Status:needs work» needs review

Updated.

AttachmentSize
feedapi-more_coder_format.patch 9 KB

#4

Status:needs review» fixed

Thanks, it's committed for both 5.x and 6.x branches. I also run through coder module again in the two codebases and i fixed the warnings where it's possible.
Btw:

$a = $b. $c;

How to write this to avoid coder module complain? :)

#5

<?php
$a
= $b . $c;
?>

#6

#7

Status:fixed» closed (fixed)

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

nobody click here