We need to get the Injector up and running in Drupal 7. So we need to create a D7 code branch.

Comments

hswong3i’s picture

What if we rename this module as "Content Injector (content_injector)" or "Ads Injector (ads_injector)"? We may port all code to D7 with a new name, similar as:

I would like to give a hand in this new version, too ;-)

mcurry’s picture

What if we rename this module as "Content Injector (content_injector)" or "Ads Injector (ads_injector)"? We may port all code to D7 with a new name, similar as:

http://drupal.org/project/globalredirect => http://drupal.org/project/redirect
http://drupal.org/project/nodewords => http://drupal.org/project/metatags

I would like to give a hand in this new version, too ;-)

I've requested info on renaming the project machine name (I can't find any UI for doing it).
See: #1126224: Help change a project's 'short project name'? in Git (and D.O. projects directory).

I'm awaiting a response.

I favor Content Injector (content_injector) because you can already use the module to inject text, html, JavaScript, etc. With some mods, it could be very powerful and general-purpose. More on this later!

Thanks for your offer to help. I can grant you access when we're ready to proceed with serious development.

mcurry’s picture

I have begun the port to D7, I'll create a D7 branch as soon as I have something working.

likewhoa’s picture

Title: Create Drupal 7 Code Branch and initial port. » Port AdSense Content Injector to Drupal 7
Issue tags: +port to d7. d7 porting

subscribing

Nikdilis’s picture

+1

marcoka’s picture

subscribe

mcurry’s picture

It's next on my list, folks. Stay tuned.

himagarwal’s picture

Subscribing.

NickLitten’s picture

/me eagerly awaiting D7 version. ;)

mcurry’s picture

@FinkPad:

I have a D7 version up and running, it has a few 'anomalies' -- the config menu entry is wrong somehow, so you don't see the settings link where it belongs. So I need to fix that...

I may commit a -dev D7 branch within the next few days... stay tuned.

carusen’s picture

Another person eagerly waiting for the D7 version. :D

samgreco’s picture

+1

kepesv’s picture

subscribe

vodsdarov’s picture

subscribing

NickLitten’s picture

I'm happy to help with testing.... just let me know ;)

mcurry’s picture

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

I've created the 7.x-1.x branch, and commited the first revision of the D7 port. If all goes well, we should see a 7.x-1.x-dev release on the home page within the next 12 hours.

Here's the project page: http://drupal.org/node/1250498

Stay tuned...

mcurry’s picture

Title: Port AdSense Content Injector to Drupal 7 » Create Drupal 7 branch, commit first working version, and generate -dev release.
Status: Needs review » Fixed

Marking as 'fixed' since the task as described in the Issue Summary is complete.

If you have any issues with the D7 branch, please create a new issue; do not re-open this issue or post additional comments to this issue.

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