Ugh, there's lots of crappy stuff going on about when to save the XML sitemap data from the user account form and when to save it because the $account object changes. I'm looking into this right now, but currently the per-user settings are not working on 7.x-2.x or 6.x-2.x.

Comments

dave reid’s picture

Issue tags: +Release blocker

Adding release blocker tag.

Anonymous’s picture

I don't know about 7 but for 6 it is pretty darn ugly. When the 'form' is loaded set a static cache of the account removing the form_build_id from the object, unserialize the object->$data and remove the form_build_id element from the array. On the 'after_update' move the $account variable to another variable in order to do the dance with form_build_id so that you can compare the before image and after image copies to know if you need an update. You have to remove the form_build_id because the values are different in the before and after.

HTH

dave reid’s picture

Priority: Normal » Major
dave reid’s picture

I think #786484: Use a 'link update' queue for delayed link changes is what we'll need to fix this... that way we get the fully changed user account variable at the end of the page request or during cron.

dave reid’s picture

Status: Active » Postponed
Anonymous’s picture

Priority: Major » Critical

I've added a note to Known Issues that will need removed once this is corrected.

Anonymous’s picture

Assigned: dave reid »
Status: Postponed » Postponed (maintainer needs more info)

Dave, I've been trying to determine why xmlsitemap_user module isn't saving the changes to include or exclude the user in the sitemap. I see that xmlsitemap_process_form_link_options() is added to the #submit array and it is called, however that function doesn't do the save. Instead it is waiting for xmlsitemap_user_user_update() hook implementation to do the save but the $account variable doesn't contain the changed value and neither does the $edit variable.

Is there a reason not to save the data on submit? What purpose is accomplished in waiting for the hook_*_update() function? I realize if we refactor this piece that we must do so for all entities.

webservant316’s picture

Issue summary: View changes

status update? is the user sub-module still off limits?

Leeteq’s picture

Leeteq’s picture

The issue that held up
#786484: Use a 'link update' queue for delayed link changes
was fixed 4 years ago, and one year ago a patch to that one was posted tentatively, which is still awaiting review / feedback.

Leeteq’s picture

Status: Postponed (maintainer needs more info) » Active
webservant316’s picture

Is the the user sub-project still unusable? I saw there was a new release, but I didn't think I saw any patched related to this issue? I would like to use the user sub-module if available.

webservant316’s picture

still no patch?

VIPIN M P’s picture

Version: 7.x-2.x-dev » 7.x-2.4
Assigned: » Unassigned

Hi..

Can any one have update the individual user exclude related work with sitemap ?