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

Canonical URL meta tag support #1648

Closed
inureyes opened this issue Feb 27, 2014 · 7 comments
Closed

Canonical URL meta tag support #1648

inureyes opened this issue Feb 27, 2014 · 7 comments

Comments

@inureyes
Copy link
Member

Summary

  • Supports canonical URL meta tag support
  • For search engines.
@inureyes inureyes added this to the 1.9.0 milestone Feb 27, 2014
@inureyes inureyes self-assigned this Feb 27, 2014
@inureyes
Copy link
Member Author

@inureyes
Copy link
Member Author

inureyes commented Mar 6, 2014

d7a9485

@inureyes
Copy link
Member Author

inureyes commented Mar 6, 2014

a3439f3

@inureyes
Copy link
Member Author

inureyes commented Mar 6, 2014

Do not know why, but automatic code ticket matching is not working on this issue. (-)

@inureyes
Copy link
Member Author

inureyes commented Mar 7, 2014

For some cases, double slashes are appeared. (e.g. http://example.org//123 )

@inureyes inureyes reopened this Mar 7, 2014
@pat-al
Copy link

pat-al commented Mar 8, 2014

framework/model/URIHandler.php 181째 줄
$this->uri['permalink'] = rtrim($this->uri['default'].$this->suri['directive'].(empty($this->suri['id']) ? '/'.$this->suri['value'] : (substr($this->suri['directive'],-1)=='/' ? '' : '/').$this->suri['id']),'/');

이렇게 substr 함수 끼워 넣어 쓰고 있습니다. 숫자 글 주소 쓸 때만 $this->suri['directive']에 빗금이 덧붙는 듯 합니다.

@inureyes
Copy link
Member Author

inureyes commented Mar 9, 2014

@pat-al 👍

inureyes added a commit that referenced this issue Mar 9, 2014
 - Modified version of pat-al's solution
   ( #1648 )
inureyes added a commit that referenced this issue Mar 9, 2014
 - Modified version of pat-al's solution
@inureyes inureyes closed this as completed Mar 9, 2014
inureyes added a commit that referenced this issue Mar 18, 2014
 - Modified version of pat-al's solution
   ( #1648 )
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants