Needs work
Project:
Context UUID
Version:
7.x-1.0-beta2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
3 Nov 2014 at 15:06 UTC
Updated:
14 May 2015 at 13:34 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
eyilmazComment #2
eyilmazComment #3
eyilmazreferencing the same issue from context module, maybe the way the issue solved there should be applied also here (instead of the attached patch).
Comment #4
skwashd commentedIn general this looks good. I found one formatting issue - see below.
This needs to comply with the coding standards, namely the you need the braces and the
$termassignment needs to be on a new line.Comment #5
eyilmazHi,
thanks for reviewing.
Here is the patch with braces.