Ctools adds a uuid_is_valid() function, which also exists in skinr.uuid.inc. We need to check for it before declaring it.

CommentFileSizeAuthor
#1 skinr-ctools_uuid-2253493-1.patch826 bytesmoonray

Comments

moonray’s picture

moonray commented
Status: Active » Needs review
StatusFileSize
new skinr-ctools_uuid-2253493-1.patch826 bytes

This patch wraps our function in an if block to avoid conflicts with Ctools' and PECL's uuid_is_valid().

moonray’s picture

moonray commented
Status: Needs review » Fixed

  • Commit 230302c on 7.x-2.x by moonray: 
    Issue #2253493 by moonray: Fixed Ctools update 7.x-1.4 breaks tests.

Status: Fixed » Needs work

The last submitted patch, 1: skinr-ctools_uuid-2253493-1.patch, failed testing.

moonray’s picture

moonray commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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