Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Call modResource refreshURIs() when clearing site cache #9129

Closed
pixelchutes opened this issue Nov 17, 2012 · 2 comments
Closed

Call modResource refreshURIs() when clearing site cache #9129

pixelchutes opened this issue Nov 17, 2012 · 2 comments
Labels
feature Request about implementing a brand new function or possibility.

Comments

@pixelchutes
Copy link
Contributor

pixelchutes created Redmine issue ID 9129

Many times it can be a huge time-saver to edit the MODx database directly, specifically if you're working with multiple MODx instances and need to copy resources between databases, or creating resources via SQL directly.

Since Revolution, the uri column on modx_site_content makes this process slightly more difficult, as the uri for reach resource must be regenerated, especially if the alias or hierarchy has been altered.

Is there any implication from adding a refreshURIs() call for modResource as part of clearing the site cache? This would simplify this process for quite a few users and prevent some of the current work-arounds required to make this process as easy as it could be.

More info here:
http://forums.modx.com/thread/80495/edited-alias-but-front-end-still-using-old-alias

@pixelchutes
Copy link
Contributor Author

pixelchutes submitted:

I have submitted the following PR: #442

This seems to take care of the problem of desynchronized URIs as a result of Evo migrations (e.g. empty uri field), direct database edits against alias / hierarchy, etc. It places the automatic uri generation call in the Clear Cache command, which most people presumably try while troubleshooting.

theboxer added a commit that referenced this issue Feb 28, 2014
* origin/pr/442:
  [#9129] Add refreshURIs call as part of clearing the site cache
@theboxer
Copy link
Member

Should be resolved by #442

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Request about implementing a brand new function or possibility.
Projects
None yet
Development

No branches or pull requests

2 participants