Ticket #1146 (assigned enhancement)

Opened 7 weeks ago

Last modified 6 weeks ago

지역로그 로컬 추천 기능

Reported by: inureyes Owned by: creorix
Priority: major Milestone: 1.7.7
Component: API Version: 1.7
Keywords: locallog Cc:
Release:

Description

개요

Change History

follow-up: ↓ 2   Changed 6 weeks ago by creorix

resources/script/locationtag.js를 뜯어보니 resources/script/generaltag.js와는 다르게 로컬 정보를 받아오는 부분이 전혀 없네요. 버그는 아닌 것 같습니다. 혹시 기능 추가가 필요하시다면 다시 말씀해주세요 :)

in reply to: ↑ 1   Changed 6 weeks ago by inureyes

Replying to creorix:

resources/script/locationtag.js를 뜯어보니 resources/script/generaltag.js와는 다르게 로컬 정보를 받아오는 부분이 전혀 없네요. 버그는 아닌 것 같습니다. 혹시 기능 추가가 필요하시다면 다시 말씀해주세요 :)

예 그렇다면 추가를! 부탁드립니다~

  Changed 6 weeks ago by creorix

  • status changed from new to assigned
  • type changed from defect to enhancement

  Changed 6 weeks ago by creorix

[7048]

refs #1146

  • 로컬 지역로그 추천 기능 추가

follow-up: ↓ 6   Changed 6 weeks ago by creorix

지역 태그 추천 부분에서 거의 가져온 코드라서 오류가 있을 수 있습니다. 테스트 부탁드립니다 :)

in reply to: ↑ 5   Changed 6 weeks ago by creorix

Replying to creorix:

지역 태그 추천 부분에서 거의 가져온 코드라서 오류가 있을 수 있습니다. 테스트 부탁드립니다 :)

지역 태그 추천이 아니라 로컬 태그 추천입니다. 헷갈렸네요.

p.s. SQL expression을 GET으로 직접 넘기는 것이 저로서는 영 꺼림직합니다만, 안 고쳐도 상관없을까요? (mysql_query가 multiple query를 지원하지 않으니 SQL Injection으로 큰 문제가 발생할 것 같지는 않습니다만....)

  Changed 6 weeks ago by inureyes

돌려본 결과, 현재 레벨에 상관없이 (1단, 2단 등등...) 언제나 첫번째를 기준으로 전체 태그가 제안되는 문제가 있는 듯 합니다. 지역이 저장된 깊이에 따라 (대한민국/서울시/광진구 라면, 첫번째 로그를 입력할 때는 대한민국만, 두번째의 경우 서울시만 제안되는 식으로) 추천해 주어야 할 것 같습니다.
덧붙여, SQL을 넘기는 부분은 수정하는게 좋겠네요

  Changed 6 weeks ago by creorix

[7084]

refs #1146

  • 지역로그 추천에서 depth가 고려되지 않은 문제 수정
  • 지역로그, 태그 추천에서 조회에 raw SQL을 사용하는 문제 수정
Note: See TracTickets for help on using tickets.