I don't think the fixes for the 1.x version have been implemented onto 2.x version.

Comments

darren oh’s picture

Status: Active » Closed (fixed)

They definitely have. Please re-open if you have specific information.

Crimson’s picture

Status: Closed (fixed) » Active

If the fixes have been implemented, then it's still not working properly. I'm still getting intermittent links in the sitemap that are displayed as "node/#" instead of their path aliases made by Pathauto.

One thing though, a clean install of XML Sitemap does not create a perfect sitemap like what some people mention in the other similar issue.

I'm using the 11/01/07 version of XML Sitemap and Pathauto version 2.0-beta4.

darren oh’s picture

Status: Active » Closed (duplicate)

Duplicate of issue 187451.

Crimson’s picture

Status: Closed (duplicate) » Active

My tables are MyISAM. It's not the same problem.

Some of the links in the sitemap are showing as their pathauto aliases and some are not.

darren oh’s picture

Status: Active » Postponed (maintainer needs more info)

As you can see in the other issue, the module works for sites with 10,000 - 30,000 nodes. How many does your site have? If you re-save one of the nodes, does it then appear under its alias?

Crimson’s picture

But that's dev-1.x, I'm using dev-2.x. Anyways, you're probably a better judge of that.

I don't have as many, only ~200 for now. If I resave, it still doesn't use the url alias in the sitemap. It seems pretty systematic, the nodes that already have the correct url aliases won't switch to "node/#" if I resave and keeps its correct alias. And the nodes that don't work won't change to their correct url aliases no matter what I do.

And like I've said before, I've uninstalled the module many times and reinstalled it just as many, and the sitemap always comes out the same, the same exact nodes have the right aliases and the same exact nodes have the wrong aliases.

darren oh’s picture

I wrote the code for both only recently. The queries are identical. Sounds like a module weight problem. Figure out which modules are changing the URL aliases, then install Module Weight and give XML Sitemap a heavier weight than those modules.

darren oh’s picture

Project: Google Sitemap » XML sitemap
Version: 5.x-2.x-dev » 5.x-1.0
Component: Code » xmlsitemap_node
Crimson’s picture

I'll try out Module Weight. And I think only Pathauto's changing my URL aliases.

blb’s picture

I'm having the same problem with a very simple site - few nodes.

changing the weight doesn't work. reinstalling doesn't work. (i've tried the dev 2 branch and the 5.x-1.0 release)

strangely, the xmlsitemap_node table is empty, but the sitemap does get generated - only with node/xx URLs.

Crimson’s picture

Oh wow. I just checked the xmlsitemap_node table and I see a big problem. *still haven't tried Module Weigh yet though*

Here's an except from the table:

+------------+------+--------------+--------------------+--------------+------------------+-------------------+
| nid        | pid  | last_changed | previously_changed | last_comment | previous_comment | priority_override |
|         41 |  377 |   1194096777 |         1194096777 |   1194284196 |       1194097942 |              NULL |
|         44 |  307 |   1194099929 |               NULL |   1194193876 |       1194193489 |              NULL |
|         52 |  333 |   1194138491 |               NULL |   1194307464 |       1194304930 |              NULL |
|         56 |  368 |   1194141068 |               NULL |   1194150220 |       1194149011 |              NULL |
| 1194709097 | NULL |            0 |               NULL |         NULL |             NULL |               187 |
|         65 |  349 |   1194151455 |               NULL |   1194283372 |       1194226998 |              NULL |
|         66 |  350 |   1194151827 |               NULL |   1194289204 |       1194217724 |              NULL |
|         72 |  776 |   1194316077 |               NULL |   1194251531 |       1194251451 |              NULL |
|         73 |  475 |   1194226026 |               NULL |   1194283585 |       1194230522 |              NULL |
| 1194806707 | NULL |            0 |               NULL |         NULL |             NULL |               260 |
|         77 |  418 |   1194289387 |               NULL |   1194285497 |       1194279327 |              NULL |
|         78 |  438 |   1194195176 |               NULL |   1194303463 |       1194301866 |              NULL |
|         80 |  440 |   1194196797 |               NULL |   1194308846 |       1194292196 |              NULL |
|         82 |  483 |   1194231328 |               NULL |   1194303412 |       1194253795 |              NULL |
|         87 |  467 |   1194223676 |               NULL |   1194298301 |       1194232355 |              NULL |
| 1194803891 | NULL |            0 |               NULL |         NULL |             NULL |               259 |
| 1194389351 | NULL |            0 |               NULL |         NULL |             NULL |               137 |
|         94 |  510 |   1194271143 |               NULL |   1194300501 |       1194277472 |              NULL |
|         96 |  516 |   1194276039 |               NULL |   1194326562 |       1194311862 |              NULL |
|         97 |  517 |   1194278097 |               NULL |   1194325127 |       1194283246 |              NULL |
|         98 |  522 |   1194279644 |               NULL |   1194312778 |       1194301225 |              NULL |
|         99 |  523 |   1194279868 |               NULL |   1194309085 |       1194308480 |              NULL |
|        101 |  529 |   1194281600 |               NULL |   1194308718 |       1194302782 |              NULL |
|        102 |  530 |   1194283046 |               NULL |   1194320156 |       1194298056 |              NULL |
|        104 |  551 |   1194306609 |               NULL |   1194326450 |       1194325939 |              NULL |
|        109 |  545 |   1194303917 |               NULL |   1194325641 |       1194323515 |              NULL |
|        112 |  550 |   1194305205 |               NULL |   1194323677 |       1194323328 |              NULL |
|        113 |  555 |   1194309562 |               NULL |   1194324963 |       1194309230 |              NULL |
|        117 |  779 |   1194318161 |               NULL |   1194326673 |       1194322269 |              NULL |
|        118 |  781 |   1194322054 |               NULL |   1194325733 |       1194322856 |              NULL |
| 1194334928 | NULL |            0 |               NULL |         NULL |             NULL |               121 |
| 1194335034 | NULL |            0 |               NULL |         NULL |             NULL |               122 |
| 1194335173 | NULL |            0 |               NULL |         NULL |             NULL |               123 |
| 1194345078 | NULL |            0 |               NULL |         NULL |             NULL |               124 |
| 1194345520 | NULL |            0 |               NULL |         NULL |             NULL |               125 |
| 1194345643 | NULL |            0 |               NULL |         NULL |             NULL |               126 |
| 1194346218 | NULL |            0 |               NULL |         NULL |             NULL |               127 |
| 1194346832 | NULL |            0 |               NULL |         NULL |             NULL |               128 |
| 1194347791 | NULL |            0 |               NULL |         NULL |             NULL |               129 |
| 1194347929 | NULL |            0 |               NULL |         NULL |             NULL |               130 |
| 1194353231 | NULL |            0 |               NULL |         NULL |             NULL |               131 |
| 1194354193 | NULL |            0 |               NULL |         NULL |             NULL |               132 |
| 1194359617 | NULL |            0 |               NULL |         NULL |             NULL |               133 |
+------------+------+--------------+--------------------+--------------+------------------+-------------------+

Is it me or are some of those entries backwards? And the one that have the right correct entries are the one that are showing up with the right alias in the sitemap.

Oh yeah. I also get this error in the watchdog A LOT (like every minute). It might be related.

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE uid = 304' at line 3 query: UPDATE xmlsitemap_user SET pid = 1602, last_changed = 1194814768, previously_changed = last_changed, priority_override = WHERE uid = 304 in /home/name/public_html/includes/database.mysql.inc on line 163.

My setup is MySQL 5.x and PHP 5.x.

darren oh’s picture

Title: Doesn't use aliases created by Pathauto for 2.x » Doesn't use aliases created by Pathauto
Status: Postponed (maintainer needs more info) » Fixed

Fixed in CVS commit 87678.

darren oh’s picture

Title: Doesn't use aliases created by Pathauto » Values in wrong order
Crimson’s picture

Status: Fixed » Active

I seem to be getting a different kind of error intermittently now.

I don't get "values in the wrong order" but it looks kind of like the same error so I'll post it here.

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'FROM node n LEFT JOIN url_alias ua ON ua.src = CONCAT('node/', 401) ' at line 3 query: INSERT INTO xmlsitemap_node (nid, pid, last_changed, priority_override) SELECT 401, ua.pid, 1194925942, FROM node n LEFT JOIN url_alias ua ON ua.src = CONCAT('node/', 401) LIMIT 0, 1 in /home/name/public_html/includes/database.mysql.inc on line 163.

I can post another issue for this if you want.

darren oh’s picture

Status: Active » Fixed

Similar errors are reported in issue 191723.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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