Closed (fixed)
Project:
XML sitemap
Version:
5.x-1.x-dev
Component:
xmlsitemap_engines.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
2 Jun 2009 at 16:56 UTC
Updated:
12 Oct 2009 at 22:00 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dave reidThanks for the update. Fixed in 6.x-2.x with http://drupal.org/cvs?commit=219778.
Comment #2
avpadernoTo notice that the service is called Bing, now; the URL of the Live Search Webmaster Central is changed in http://www.bing.com.
It must be a change they did in the last week, as I was not able to access to Live Search Webmaster Central one week ago because the old URL was redirected to the same URL in an infinite loop.
Comment #3
Starminder commentedWill this be fixed in 6x1x?
Comment #4
dave reidHere's a patch for 6.x-1.x. Please someone test it out and verify that the upgrade works. Then I'll commit.
Comment #5
xamountwill this be backported to version 5?
Comment #6
dave reidRe-rolled from the module root.
Comment #7
dave reidxamount: Possibly. I need people to test the patch for 6.x-1.x first.
Comment #8
dave reidPatch created for 5.x-2.x.
Comment #9
CyberKieken commentedThis worked:
Message The sitemap has been successfully submitted to Bing (formerly Live Search).
Comment #10
Anonymous (not verified) commentedHere is the patch for the reset code. I will commit later today.
Comment #11
dave reidComment #12
Anonymous (not verified) commentedI must have needed some tonic.
Comment #13
dave reidRenaming title so others can find this issue easier, and marked #490178: Update Windows Live to Bing as a duplicate of this issue.
Comment #14
scoutbaker commentedThanks. I certainly didn't find it when I looked.
Comment #15
avpadernoComment #16
Anonymous (not verified) commentedCommitted to CVS.
Comment #17
dave reidPatch in #8 for 5.x-2.x should still be valid.
Comment #18
armetrix commentedWith regard to Drupal 5.x I am having a problem getting the patch to "take". I've reviewed notes and comments following all directions to the letter and still I am getting errors with regard to submitting site maps to Bing.
Comment #19
armetrix commentedWell after several attempts, we manually changed everything according to how the patch was written and voila! it works. The process wasn't too difficult. Just be certain that the submission URL points to the right place: "http://www.bing.com/webmaster/ping.aspx?siteMap=[path to your sitemap]".
Comment #20
elyobo commentedAlso having problems with the patch; see as follows.
liam@tyreal:~/drupal/modules$ !tar
tar xzf xmlsitemap-5.x-1.6.tar.gz
liam@tyreal:~/drupal/modules$ cd xmlsitemap
liam@tyreal:~/drupal/modules/xmlsitemap$ patch -p0 < 479892-D5-2.patch
patching file xmlsitemap_engines/xmlsitemap_engines.install
Hunk #1 FAILED at 27.
1 out of 1 hunk FAILED -- saving rejects to file xmlsitemap_engines/xmlsitemap_engines.install.rej
patching file xmlsitemap_engines/xmlsitemap_engines.module
Hunk #1 FAILED at 3.
Hunk #2 FAILED at 38.
Hunk #3 FAILED at 119.
Hunk #4 FAILED at 199.
Hunk #5 succeeded at 273 (offset 32 lines).
4 out of 5 hunks FAILED -- saving rejects to file xmlsitemap_engines/xmlsitemap_engines.module.rej
See attached files for more details.
Comment #21
elyobo commentedBah, ignore me; I see in the comments above that the patch is intended for the 5.x-2.x branch.
Comment #22
Anonymous (not verified) commentedIf you manage to create a patch, for 5.x-1.x-dev I can review it and commit.
Comment #23
Anonymous (not verified) commentedWe need a patch for 5.x-1.x-dev on this.
Comment #24
rkdesantos commented+1 subscribe
Comment #25
dave reidRe-rolled for 5.x-1.x. Please test and make sure running update.php works ok.
Comment #26
Anonymous (not verified) commented@elyobo or @rkdesantos: Are you working fine with the patch in #25?
Comment #27
rkdesantos commentedUnfortunately, I upgraded the D5 site I maintain to the 2.x branch because of this issue and another one. Based on Dave's answer to an inquiry I posted in the issues, the recommendation was that I not downgrade back to 1.x though 2.x is now unsupported. I'm ok either way and happy to test the patch but I can't do so unless I downgrade. So whatever the maintainers consensus is, I'll go with it.
Comment #28
dave reidTested the upgrade and it worked just fine. Committed to the 5.x-1.x branch.