Ticket #652 (closed defect: fixed)

Opened 23 months ago

Last modified 23 months ago

s_에 대한 removeAllTags의 문제

Reported by: gendoh Owned by: inureyes
Priority: major Milestone: 1.5.1
Component: Core Version: 1.5
Keywords: Cc:
Release:

Description

  • 1.1에서 s_page_title 때문에 만들었던 처리되지 않는 s_ 블럭 지우기 루틴이 완벽하지 않다.
    • 태그만 지우고 그 사이의 내용을 그대로 출력됨
    • 결국 새버전용 스킨이 구버전에 가면 내용물을 쏟아냄
  • 핸들링 할 수 없는 s_ 블럭은 내용까지 지워지는 것이 좋을 것으로 보임

Change History

follow-up: ↓ 2   Changed 23 months ago by creorix

[4374]

  • 이 정도면 될까요?

in reply to: ↑ 1 ; follow-up: ↓ 3   Changed 23 months ago by gendoh

Replying to creorix:

[4374] * 이 정도면 될까요?

ungreedy ;;;

in reply to: ↑ 2   Changed 23 months ago by creorix

Replying to gendoh:

Replying to creorix:

[4374] * 이 정도면 될까요?

ungreedy ;;;

.*? 이렇게 쓰면 ungreedy 처리 되는거 아닌가요? 방금 확인하기 위해서 돌려봤는데

<?php
$a = '<s_test>ab</s_test>cd<s_test>ab</s_test>';
echo preg_replace('@<(s_[0-1a-zA-Z_]+)>.*?</\1>@', '', $a);
?>

이렇게 하고 돌리면 cd 나오던데, 그럼 ungreedy 처리 된 것 아닌가요?;;

follow-up: ↓ 5   Changed 23 months ago by inureyes

두 분께서 결정을 내려 주시길 :)

in reply to: ↑ 4   Changed 23 months ago by creorix

Replying to inureyes:

두 분께서 결정을 내려 주시길 :)

전 QA 전문 겐도님께 판단을 맡기겠습니다 :) (책임회피가 아니에요 -_-;;)

follow-up: ↓ 8   Changed 23 months ago by gendoh

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

nested랑 element 인자 있을 때랑 등등등 있지만 쉽게 갑시다. -ㅅ-

  Changed 23 months ago by gendoh

  • status changed from closed to reopened
  • resolution fixed deleted

아 트렁크 반영 해야져.

in reply to: ↑ 6   Changed 23 months ago by creorix

Replying to gendoh:

nested랑 element 인자 있을 때랑 등등등 있지만 쉽게 갑시다. -ㅅ-

뭐 그렇긴 하지만 일단 현재의 스킨 스펙 안에서 같은 태그가 안에 또 나오는 경우가 없기 때문에 괜찮을 것 같습니다 :) (혹시 처리를 원하신다면 티켓 만들어두고 mileston 2.0으로 잡아두시면 천천히 처리하겠습니다~)

  Changed 23 months ago by inureyes

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

[4383]

  Changed 23 months ago by gendoh

  • status changed from closed to reopened
  • resolution fixed deleted

아차차.. PCRE_DOTALL

follow-up: ↓ 12   Changed 23 months ago by gendoh

그리고 0-1;;;;; 0-9겠죠?

in reply to: ↑ 11   Changed 23 months ago by creorix

Replying to gendoh:

아차차.. PCRE_DOTALL 그리고 0-1;;;;; 0-9겠죠?

[4386]입니다. 생각없이 커밋하면 이렇게 되는군요 -_- 반영 부탁드립니다.

  Changed 23 months ago by inureyes

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

[4391]

Note: See TracTickets for help on using tickets.