Ticket #746 (reopened defect)

Opened 12 months ago

Last modified 8 months ago

1-depth 카테고리의 이름에 /가 있으면 발생하는 문제

Reported by: creorix Owned by: inureyes
Priority: major Milestone: 1.5.4
Component: Core Version: 1.5
Keywords: category Cc: dev@…
Release:

Description

  • depth가 1인 카테고리의 이름에 슬래시가 있으면 나중에 해당 카테고리의 글을 검색할 때 문제가 생깁니다. (label이 슬래시로 구분되어 있기 때문)
  • 재현 방법
    • 카테고리 a를 만들고 그 아래에 카테고리 b를 만든다.
    • a/b라는 이름의 카테고리를 만든다.
    • 두 가지 링크를 모두 클릭시 한 쪽으로만 이동하는 것을 확인할 수 있다.

Change History

  Changed 12 months ago by inureyes

  • status changed from new to assigned

이게 그러니까 글 이름 (entry)이 카테고리 이름과 겹치는 경우인 것인가요?

  Changed 12 months ago by creorix

카테고리 이름끼리 겹치는 경우입니다. a라는 카테고리의 하위에 b라는 카테고리가 있는 경우와 a/b라는 카테고리가 있는 경우에 두 페이지의 URL이 같기 때문에 발생하는 문제입니다.

  Changed 12 months ago by inureyes

그러면 카테고리 이름에 /을 사용하지 못하도록 해야 하는 부분이겠군요...

follow-up: ↓ 11   Changed 12 months ago by creorix

[4732]

  • 일단 inureyes님의 말처럼 /를 카테고리 이름으로 사용할 수 없도록 했습니다.
  • 저장시 urlencode해서 저장하는 방법도 있는데 그러면 suri와 관련된 부분 전부를 뜯어고쳐야 되는 문제가 되기에 -_-;;
  • 반영 부탁드립니다.

follow-up: ↓ 6   Changed 12 months ago by lunamoth

기존 분류에 "/" 슬래쉬를 사용하고 있던 유저들은 분류를 변경을 해야되는 것인지요?

in reply to: ↑ 5   Changed 12 months ago by creorix

Replying to lunamoth:

기존 분류에 "/" 슬래쉬를 사용하고 있던 유저들은 분류를 변경을 해야되는 것인지요?

사실 그것이 문제입니다.
현 상태에서는 사용하는 데에 문제는 없지만 변경이 불가능합니다. 하지만 URL 상으로도 문제가 있는 부분이라서 그냥 둘 수 있는 문제는 아니고, 그렇다고 checkup시 강제로 삭제할 수도 없습니다.
가장 좋은 방법은 label에 저장할 때 rawurlencode를 이용해서 각 segment를 인코드하는 것입니다. 그런데 문제는 $surivalue?가 이미 디코드된 스트링을 가지고 있는다는 데에 있습니다. 이를 비교하려면 인코드된 스트링이 필요한데 suri 변수는 이를 가지고 있지 않은 것 같습니다. 이 때문에 해당 부분을 뜯어고치려면 관련 부분에 전부 처리를 해 주어야 하는 문제가 발생합니다.
다른 방법은 없을까요?

  Changed 12 months ago by inureyes

[4736]

  • [4732] 반영
  • 일단 이 부분은 슬래시를 받지 못하도록 하는 것이 이후의 parsing 에도 유리할 것 같습니다.

  Changed 12 months ago by inureyes

  • status changed from assigned to closed
  • resolution set to fixed

[4746]

  • trunk에서 1.5 트리로의 반영

  Changed 10 months ago by lunamoth

r5123 분류 생성이 아닌 분류 이름 변경을 통해서는 슬래시를 사용할 수 있는듯 싶습니다.

  Changed 9 months ago by creorix

  • status changed from closed to reopened
  • resolution deleted

[5619]

  • 분류 이름 변경시 금칙 문자 체크가 이루어지지 않는 문제 수정

in reply to: ↑ 4   Changed 8 months ago by gendoh

Replying to creorix:

[4732] * 일단 inureyes님의 말처럼 /를 카테고리 이름으로 사용할 수 없도록 했습니다. * 저장시 urlencode해서 저장하는 방법도 있는데 그러면 suri와 관련된 부분 전부를 뜯어고쳐야 되는 문제가 되기에 -_-;; * 반영 부탁드립니다.

#892 : 어차피 뜯어 고쳐야 할 거 같아요

Note: See TracTickets for help on using tickets.