Eliminate coder module warnings

mustafau - March 29, 2008 - 15:40
Project:FeedAPI
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

Patch is attached.

AttachmentSize
feedapi_coder_warnings.patch4.6 KB

#1

Aron Novak - April 4, 2008 - 20:14
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

mustafau - April 9, 2008 - 15:21
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

mustafau - April 10, 2008 - 16:09
Status:needs work» needs review

Updated.

AttachmentSize
feedapi-more_coder_format.patch 9 KB

#4

Aron Novak - April 11, 2008 - 19:49
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

mustafau - April 12, 2008 - 12:22

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

#6

mustafau - April 12, 2008 - 12:42

#7

Anonymous (not verified) - April 26, 2008 - 12:52
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.