Create a Drupal 7 version [#1178316]

Comment	File	Size	Author
#3	d7versionofkiva-1178316-3.patch	16.18 KB	coderintherye
#1	d7versionofkiva-1178316.patch	14.57 KB	coderintherye

Comment #1

coderintherye CreditAttribution: coderintherye commented 6 June 2011 at 20:01

File	Size
d7versionofkiva-1178316.patch	14.57 KB

Here's a patch against D6 latest but also requires my patch from http://drupal.org/node/504942#comment-4556072, going to see if I can commit a D7 release or if I will need CrookedNumber to do it.

Note this is only an upgrade of the Kiva module. I haven't yet done an upgrade of the KivaFeeds module.

Log in or register to post comments

Comment #2

coderintherye CreditAttribution: coderintherye commented 6 June 2011 at 20:01

Status:

Active

» Needs review

Log in or register to post comments

Comment #3

coderintherye CreditAttribution: coderintherye commented 6 June 2011 at 20:07

File	Size
d7versionofkiva-1178316-3.patch	16.18 KB

Updating patch cause two of the hunks didn't apply, so instead this patch just goes directly off the current 6.x.-1.x release (no need to apply the other patch).

Log in or register to post comments

Comment #4

coderintherye CreditAttribution: coderintherye commented 6 June 2011 at 20:16

Assigned:

coderintherye

» CrookedNumber

Ok I created a dev branch for D7 as per the git-instructions and you can see my commit here: http://drupal.org/commitlog/commit/8700/86a046fd06f687fd7efa6944eb214f19...

However, I don't have permissions to create a release, so give it a review when you have a chance and let me know :)

To note, I'm running the D7 version on a dev machine and tested it out with lenders and partners and such, but haven't tested all features yet, so far so good though.

Log in or register to post comments

Comment #5

CrookedNumber CreditAttribution: CrookedNumber commented 13 June 2011 at 01:41

Just gave you perm to add releases. Let 'er rip!

Log in or register to post comments

Comment #6

CrookedNumber CreditAttribution: CrookedNumber commented 13 June 2011 at 01:56

Looking good. Installs cleanly. Getting a bunch of notices about undefined vars -- but that's par for the course (PHP5!).

Log in or register to post comments

Comment #7

CrookedNumber CreditAttribution: CrookedNumber commented 13 June 2011 at 02:00

FYI: Also created #1186610: Create a Drupal 7 version for kivafeeds and assigned it to CrookedNumber.

Log in or register to post comments

Comment #8

coderintherye CreditAttribution: coderintherye commented 13 June 2011 at 04:39

Coolness :)

I just added a release for 7.x, if I remember correctly I will need to come back publish the release after it goes through the system, takes a bit of time for dev releases. I'll mark this as resolved once I come back to it.