Include ExplorerCanvas library via-LibrariesAPI [#849232]

Comment	File	Size	Author
#15	include-excanvas-via-libraries-api-d7-849232-15.patch	5.24 KB	dharizza
#13	include-excanvas-via-libraries-api-d7-849232-13.patch	5.18 KB	acouch
#11	849232-beautytips-libraries-api.patch	1.77 KB	mrfelton
#9	README.patch	938 bytes	sleeping_at-work-
#2	beautytips-librariesAPI_support.patch	586 bytes	sleeping_at-work-
#1	beautytips-librariesAPI.patch	411 bytes	sleeping_at-work-

Comment #1

sleeping_at-work- CreditAttribution: sleeping_at-work- commented 9 July 2010 at 10:54

Version:	6.x-2.0	» 6.x-2.x-dev
Status:	Active	» Needs review

File	Size
beautytips-librariesAPI.patch	411 bytes

Would it be possible to include the ExplorerCanvas library through the Drupal Libraries API http://drupal.org/project/libraries ? This will bring all the advantages of the Libraries API. I'm adding a patch which needs to be tested.

Log in or register to post comments

Comment #2

sleeping_at-work- CreditAttribution: sleeping_at-work- commented 9 July 2010 at 15:23

File	Size
beautytips-librariesAPI_support.patch	586 bytes

I'm adding a new patch which is aimed just to support the Drupal Libraries API, without compromising the usability of the module for who's not implementing Libraries API in his system. It lacks testing.

Log in or register to post comments

Comment #3

kleinmp CreditAttribution: kleinmp commented 9 July 2010 at 17:13

Thanks for the patch!

I didn't know about the Drupal Libraries API, and it's a good idea. I'll test out the patch and commit it the next chance I get.

Log in or register to post comments

Comment #4

intyms CreditAttribution: intyms commented 10 July 2010 at 11:17

subscribing...

Log in or register to post comments

Comment #5

sleeping_at-work- CreditAttribution: sleeping_at-work- commented 12 July 2010 at 16:35

thanks :)

Log in or register to post comments

Comment #6

Anonymous (not verified) CreditAttribution: Anonymous commented 24 July 2010 at 01:21

subshcribing.

Log in or register to post comments

Comment #7

kleinmp CreditAttribution: kleinmp commented 6 August 2010 at 21:50

Status:

Needs review

» Fixed

Committed to the dev version.

Log in or register to post comments

Comment #8

20 August 2010 at 22:00

Status:

Fixed

» Closed (fixed)

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

Log in or register to post comments

Comment #9

sleeping_at-work- CreditAttribution: sleeping_at-work- commented 31 January 2011 at 12:24

Status:

Closed (fixed)

» Needs review

File	Size
README.patch	938 bytes

In order to grant consistency, please consider to review the README
Here's a patch

Log in or register to post comments

Comment #10

mrfelton CreditAttribution: mrfelton commented 18 May 2011 at 18:49

Status:

Needs review

» Needs work

This patch is not in the latest 2.x-dev branch.

Log in or register to post comments

Comment #11

mrfelton CreditAttribution: mrfelton commented 18 May 2011 at 18:54

Status:

Needs work

» Needs review

File	Size
849232-beautytips-libraries-api.patch	1.77 KB

Here is a patch that reincorporates this fix as well as the update to the README.txt file into the latest 2.x-dev branch

Log in or register to post comments

Comment #12

SocialNicheGuru CreditAttribution: SocialNicheGuru commented 7 February 2012 at 13:39

Version:

6.x-2.x-dev

» 7.x-2.x-dev

Will this be ported to D7?

Log in or register to post comments

Comment #13

acouch CreditAttribution: acouch commented 26 July 2013 at 14:37

File	Size
include-excanvas-via-libraries-api-d7-849232-13.patch	5.18 KB

Here is a D7 patch. This was necessary for us because we are trying to include this in a distro which makes it hard to package the excanvas library in the beautytips module and the "WARNING!" message is fired upon install which makes it look like the install was unsuccessful.