E_ALL compilance

Yorirou - July 1, 2009 - 11:22
Project:User Relationships
Version:6.x-1.0-beta10
Component:Code
Category:bug report
Priority:normal
Assigned:alex.k
Status:closed
Description

I got two E_NOTICE in user_relationships_node_access.
I attached the patch.

AttachmentSize
user_relationships_node_access.patch814 bytes

#1

alex.k - July 1, 2009 - 12:27
Assigned to:Anonymous» alex.k
Status:active» fixed

Committed, thanks!

#2

Yorirou - July 1, 2009 - 12:37
Status:fixed» active

Thx!

I have two more patches.

In some functions, completely undefined (I haven't found any sign of these variables elsewhere) variables called. Are these logical bugs, or just missing features?

AttachmentSize
user_relationships_api.api_.inc_.patch 1.17 KB
user_relationships_api.module.patch 529 bytes

#3

Yorirou - July 1, 2009 - 12:56

And another one patch.

AttachmentSize
user_relationships_node_access.module.patch 1003 bytes

#4

alex.k - July 1, 2009 - 14:56

Some of these were fixed recently, could you recheck them against current CVS or the dev release? Thank you.

#5

alex.k - July 1, 2009 - 16:21
Status:active» needs work

#6

alex.k - July 5, 2009 - 21:39
Status:needs work» fixed

Checked myself and committed http://drupal.org/cvs?commit=233882. Thanks again!

#7

System Message - July 19, 2009 - 21:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.