Drupal 7 version of FeedBurner

Yes it's definitely getting ported at some point soon. I've been working on stabilizing the Redirect API which the FeedBurner module will depend on.

subscribe

Drupal 7.0-rc1 out

I'm aware. Stop mass bumping issues.

+1

subscribe

subscribe

I like the idea of a unified common module, but for the moment that means no feedburner module for D7. That means I can't port my site, and I'm sure there are many others who depend on this module.

Just complaining or asking when will this be ready doesn't help.

If there is something you can do about this (code, resources, etc) then by all means express it, but if you just say, "I want this now", no one will listen.

subscribing

subscribe

subscribe

subscribing

+1 =)

subs

subs +1

subscribe

Initial port, not using the Redirect module for redirection yet

https://github.com/bran/feedburner

The patch is against 6.x-1.x branch

Sorry I don't think drupal.org likes that hash mark in the file name

This patch updates all the queries to d7 equivalent, and a few small updates for d7. This patch works on top of the one I posted above.

This patch works on top of the one I posted above. This patch makes "Unburn" work. Fixed all the broken links that were pointing to admin/build rather than admin/structure

Update the way xmlrpc is called to D7 way

This patch combines all changes up to this point against 6.x-1.x HEAD

includes:
#20
#21
#22
#23

Reviewed 956872-all_porting_up_to_march28-comment-24-D7.patch; looks good.

Some errors/warnings, but nothing critical, and the module still works:

1. Checked out the latest 6.x-1.x, applied patch in comment 24, enabled module and got:

DatabaseSchemaObjectExistsException: Table <em class="placeholder">feedburner</em> already exists. in DatabaseSchema->createTable() (line 630 of /includes/database/schema.inc).

2. After 'unburning' a feed:

Notice: Undefined index: unburn in feedburner_unburn_form_submit() (line 211 of /sites/all/modules/feedburner/feedburner.admin.inc).

+1

sub

+1

+1

Subscribing

subscribe

subscribe

This patch is all inclusive on 6.x-1.x HEAD, including fixes for comment #25.

my working copy is at https://github.com/bran/feedburner

The D7 version is going to be rewritten to use the APIs provided in the Redirect module, so this is not just a port.

Also waiting for this :-)

+1

+1 sub

Subs.

Subscribing for the awesomeness.

Tagging the title for those following this issue in Tracker (please allow this change—it's oh-so-helpful for people who are following about 25 other 'Drupal 7 version' or 'Drupal 7 port' issues...).

The D7 version is going to be rewritten to use the APIs provided in the Redirect module, so this is not just a port.

Cool!
What about doing a 7.x-1.x with a direct port of 6.x-1.x using the provided patches and later a 7.x-2.x using the new Redirect module API?

subscribing

subscribing

example.com/taxonomy/term/11/0/feed does not work anymore in Drupal 7.

It should be without /0 in Drupal 7.

The patch is against https://github.com/bran/feedburner.

A related issue comment of the RSS Permissions module: http://drupal.org/node/945462#comment-3938962.

@branana: Thanks for your port.

A patch for fixing some warnings:

Warnings on view:
Warning: Parameter 1 to feedburner_node_view() expected to be a reference, value given in module_invoke_all() (line 819 of /home/proj/web/dev/d7git/includes/module.inc).

fix: function feedburner_node_view($node, $view_mode = 'full', $langcode = NULL) {
reference: http://api.drupal.org/api/drupal/modules--node--node.api.php/function/ho...

Notice: Undefined property: stdClass::$in_preview in feedburner_node_view() (line 182 of /home/proj/web/dev/d7git/sites/all/modules/feedburner/feedburner.module).

fix: if (!empty($node->in_preview) && user_access('view feedflare')) {
modification idea from: http://drupalcode.org/project/location.git/blobdiff/4ad4c6d9ddc013a7d2c3...

XML RPC warnings after node edit:

Warning: key() expects parameter 1 to be array, string given in _xmlrpc() (line 557 of /home/proj/web/dev/d7git/includes/xmlrpc.inc).

Warning: Invalid argument supplied for foreach() in xmlrpc_request() (line 380 of /home/proj/web/dev/d7git/includes/xmlrpc.inc).

fix: $result = xmlrpc('http://ping.feedburner.google.com/', array('weblogUpdates.ping' => array($name, $url)));
reference: http://api.drupal.org/api/drupal/includes--common.inc/function/xmlrpc/7

The patch is against https://github.com/bran/feedburner. Patch of #45 is not included.

subscribe

Sub

Subscribe

+1 I agree with #42. Great job branana and scito!

subscribing

Subscribe

Subscribing

subscribing - we need a port for Drupal 7.

Subscribe

The patch is against https://github.com/bran/feedburner.
You can see it in action on http://drupalbridge.org/en and download the working module here http://drupalbridge.org/en/feedburner-module-drupal-7-version.

@vb Have you made any additional fixes compared to #45 and #46?

@ibex, yes

@vb... awesome, almost there. From your site, you have two Feeds. The one set up with FeedBurner works great, but the "This site (RSS)" does not when using FireFox and yahoo reader. When I try to validate your "This Site (RSS)" with http://feedvalidator.org, it throws the same errors I get. Those are:
- Missing atom:link with rel="self"
- description should not contain relative URL references (how do you change all Feed relative URLs into Absolute)

I'm getting the same errors above. FeedBurner will not accept a Feed URL that does not validate.

Can anyone comment on how to fix the two errors above in order to complete the configuration of this module?

i've tested the module linked in #56 and seems to work for me.
I've tested it with a rss view.

I've found two little problems:
- A JS error when typing in admin/structure/feedburner/burn/ (in any of the two fields)
- An error when submiting admin/structure/feedburner/burn/ and url in fedburner feed field has a ?format=html argument as recomended in the following link to show browser friendly page: http://www.google.com/support/feedburner/bin/answer.py?hl=en&answer=79738

What about a dev version that allows us better testing and issue filling?

Thanks.

Javi.-

I think that an updated dev for 7.x would be great.
We can test and create issues for correct branch.

Thanks

sub

The linked Module in #56 works fine for me on a Views-Feed.

subscribe

subscribe

No, rediresct is not enough. There could be two problems:
1. How to automatically change RSS Icon ursl in the end of page for User Foums?
2. I have 10+ Feeds on my portal and I can not create separate block for each of them...
3. Local URS: when burning feeds to feedBurner local Links not transfered automatically and they are treated as FeedBurner Local.

So let us develop something
+1

Subscribing.

Dave, I know you are extremely busy, but shouldn't you have created a 7.x-1.x branch / version (as suggested above) long time ago? Perfection is good, but not when nothing happens. For non-developers there effectively isn't a working Drupal 7 version of this module and it's soon a year since Drupal 7.0 was released. This is very sad.

I'm suggesting that some of the people contributing patches is offered to co-maintain this module.

@hansfn, I agree some could co-maintain the blog but, as you pointed out, @davereid is very busy. There is a lot of working going on with the Redirect module that is required to be finished before this module is updated.

@davereid, you rock man. Any update would not go amiss (re: redirect APIs, etc).

If you are using this module to redirect your main feed, maybe you would like to do it manually with redirect until this module is ready:

/**
 * Implements hook_menu_alter().
 */
function youcustommodule_menu_alter(&$items) {
  // Redirect to feedburner.
  $items['rss.xml']['page callback'] = 'youcustommodule_node_feed';
}

/**
 * Menu callback override for rss.xml
 *
 * Redirect to feedburner.
 */
function youcustommodule_node_feed() {
  $feedburner_url_redirect = new stdClass;
  $feedburner_url_redirect->redirect = 'http://feedproxy.google.com/yoururlhere';
  redirect_redirect($feedburner_url_redirect);
}

I want to upgrade my blog. Is there any new development?

Subscribing +1

It is beyond ridiculous that there is no Drupal 7 version out yet. 2 years? Really?

Re #77: @margokw - Dave maintains a lot of modules. Make a patch or thank him for what he did. Don't abuse. See also http://beta.openpredictionmarkets.org/node/8

sub

@intropedro and @jordan8037310 <-- Please use the big follow button in the top right corner instead of subscribe comments. Thanks!

In the the big follow button in the top right corner I see the text "Following" and if I put the mouse hover it, the text change to "Unfollow" with backbroung green.

@intropredro You started following when you added the subscribe comment. Next time do no make a subscribe comment. Just click the button, and you will follow the subject, but not notify everyone else on the comment, that you are following when you are not adding anything to the issue.

At this point, I suggest someone who has created a direct port for D7 request co-maintainership via the proper Drupal method: http://drupal.org/node/251466. The fact that this thread is the longest and most active in the issue queue for this module is the exact reason this resource is available.

Dave's desire to recreate the D7 version to rely on the Redirect API is valiant and not without purpose, but in the meantime there is no reason why a 7.x-1.x version cannot be made as a direct port. When Dave gets his version created it can be a 7.x-2.x branch, as suggested above.

I am having a lot of issues getting this module formatted correctly. Is someone willing to give me a little help?

the 2 active forks i found on github: https://github.com/ceefour/feedburner/commits/
https://github.com/bran/feedburner
Probably someone could take up the reins for @DaveReid but does anyone understand the Redirect module situation?? If it's ok now maybe we can start trying to roll some patches together.

@HongPong, I think you're over-selling the "active forks" part of your post. One of them has one commit this year, the other has two commits this year.

@ a1tsal, thanks for your link:

RewriteEngine on
RewriteCond %{HTTP_USER_AGENT} !FeedBurner
RewriteRule ^index\.xml$ http://feeds.feedburner.com/YOURFEEDADDRESS [R,L]

Does that only work for one RSS feed - what if I have numerous RSS feeds?

Thanks