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

Adding bookmark doesn't work correctly (problem with accent caracter in title) (Trac #4173) #4173

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 10 comments
Labels
Milestone

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/4173 on 41921020-05-22 by trac user duclos, assigned to unknown.

Elgg version: 1.8.1

Hello, when i try to save an adding of a bookmark with elgg bookmark plugin, Elgg redirects me on a wrong page. This is happening only when i put the accents caracters (like é or è or à or ñ).

By example, if i put a title like "tttté" i have this address :

"http://localhost/elgg-1.8.1/http://localhost/elgg-1.8.1/http://localhost/elgg-1.8.1/bookmarks/view/100/tttt"

instead of :

"http://localhost/elgg-1.8.1/bookmarks/view/100/tttt"

If i correct manualy web address it work !

@elgg-gitbot
Copy link
Author

trac user duclos wrote on 41921053-08-30

This a general problem. The same error occure with page plugin. blog plugin...

@elgg-gitbot
Copy link
Author

trac user Purus wrote on 41932007-06-23

http://community.elgg.org/pg/forum/topic/824832/strange-link-for-an-account/

@elgg-gitbot
Copy link
Author

cash wrote on 41933324-05-25

Poorly documented feature of PHP: \w is influenced by locale

@elgg-gitbot
Copy link
Author

Milestone changed to Elgg 1.8.2 by cash on 41933324-05-25

@elgg-gitbot
Copy link
Author

cash wrote on 41933370-02-26

Fixes #4173 removed use of \w since it is locale sensitive
Changeset: 2478687

@elgg-gitbot
Copy link
Author

trac user fgreinus wrote on 41937436-01-28

I implemented the Changeset from GIT ([http://trac.elgg.org/changeset/24786872cdee97470016998ff170c99b00054167] into my upgraded 1.8.1 and the error still appears.

I just added the line: $title = preg_replace("/[^A-Za-z0-9_ ]/", "", $title);

On my site those problems appear with german "umlauts", so chars like "ö,ä" etc.

@elgg-gitbot
Copy link
Author

cash wrote on 41937451-07-28

Can you provide an example? Either a link to a page on your site or a description of what the title is and what the link is

@elgg-gitbot
Copy link
Author

trac user fgreinus wrote on 41937460-03-22

For example it's in the "Last Comments" view at the blog-site (core-plugin). There it is for example the Title "Sara Könemann" and the link should be: http://domain.de/profile/K%C3%B6nemannSar (or http://domain.de/profile/KönemannSar
but it is like that:
http://domain.de/http://domain.de/profile/K%C3%B6nemannSar

@elgg-gitbot
Copy link
Author

cash wrote on 41937466-04-22

Thanks for the example. That's a different bug (but related). That bug is here: #4190

@elgg-gitbot
Copy link
Author

cash wrote on 41938425-08-02

Fixes #4173 removed use of \w since it is locale sensitive
Changeset: 2478687

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

1 participant