removeComments와 ignoreCustomFragments가 있는 버그 (& G)

4340 단어 html-minifier

묘사

image
image
이 의견의 무시 여부를 결정할 때 [텍스트]가 [reCustomIgnore]로 바뀝니다.
image
따라서 이런 논리는 영원히 재현되지 않고, 이런 논평은 삭제되며, 일부 문제를 초래할 것이다.내가 보기에 이 기능은 다음과 같은 요구에 부합해야 한다.
image
image

토론 #1

당혹스러워요. 왜 ignoreCustomComments중의 변수를 사용해야 합니까?ignoreCustomFragments@alexlamsl?이것은 ignoreCustomComments 주석을 삭제할 수 없습니다. 삭제하면 ignoreCustomComments replace back 논리 오류가 발생합니다.
나의 곤혹스러움을 참는 영혼식은땀:
html 코드(과 옵션)의 예를 제시하면 예상치 못한 결과가 나올 수 있습니까?

토론 #2

HTMLMinier 이전:
image
다음:
image
옵션:
image

토론 #셋

아, 이제 네 뜻을 알겠다.

토론 #4

가 댓글의 내용과 일치하고

토론 #5

를 사용했을 때 우리는 교체 문제에 부딪혔다.
다음 출력이 원하는 결과인지 확인해야 합니다.
{{if a==1}}<div>i'm a div</div>{{/if}}
ignoreCustomFragments@alexlamsl은 removeComments좋아요. - 한번 볼게요. +1:
@kangax라는 오류/구석 사례의 존재는

토론 #6

introduced였기 때문입니다. 당신은 이에 대해 어떤 견해를 가지고 있습니까?당신은 상술한 예상 결과에 동의합니까?

토론 #7

@kath4job은 미래에 대해 간단한 설명ignoreCustomFragments을 통해'ignoreCustomFragments'를 사용할 때 의외로 삭제

토론 #8

를 하는 것이 좋다.문제가 뭔지 알아내는 데 시간이 좀 걸렸어요...
이것은 흥미로운 용례이다. 나는 미니필드가 댓글을 삭제하는 것이 정확한 결과라고 생각한다.
{{if ... }}<p>...</p>{{/if}}
"ignoreCustomFragments"가 열려 있어도 외부 환경을 감지할 수 없습니다.만약 우리가 먼저 입력을 HTML로 해석하려고 한다면, 우리는 이러한 문제에 직면하게 될 것이다. 무시해야 할 모든 내용이 존재하기 때문에 이 표시는 효과적인 HTML이 아니기 때문에 전체 해석 과정은 지옥에 떨어질 것이다.
22조입니다.
따라서, 모든 내용을 해석하거나 축소하려고 시도하기 전에, 사용자 정의 세션을 축소의 첫 번째 단계로 무시할 수 밖에 없습니다.이후 <!-- {{ test }} -->{{if ... }}<p>...</p>{{/if}} 논리가 작용하기 시작했다.이 중 두 과정은 독립되어 서로 모르기 때문에 삭제된 부분이다.다시 말하면 <!-- {{ test }} -->가 우선이다. 왜냐하면 removeComments 이후에 운행하기 때문이다.
우리는 문서에서 이 점을 명확하게 언급해야 하지만, 나는 이것이 올바른 행위라고 생각한다.
@kath4job@alexlamsl 어때요?이게 의미가 있나요?우리는 자신의 발을 향해 총을 쏘지 않고 무엇을 할 수 있겠는가?removeComments@kangax는 예상 행위에 대해 우리의 관점은 일치한다.
너는 이미 알고 있을지 모르지만, 만일을 대비해서, 내가 분명히 말하고 싶은 것은, 이 잘못은 주석이 삭제된 것만은 아니다.이것은 다음과 같습니다.
<!--{{comment}}-->{{if}}blah{{/if}}
이렇게 됩니다.
{{comment}}blah{{if}}
순서로 바뀐 가설.
나는 이 문제를 해결할 생각이 하나 있다. (위에서 건의한 해결 방안이 우리의 예상한 결과를 내놓지 못했다.)나는 진정한 키보드로 돌아오자마자 글을 올릴 것이다.ignoreCustomFragments 네, 그렇다고 생각합니다.챙겨줘서 고마워!
내 아이패드에서 보내기

On Feb 28, 2016, at 11:45 PM, Alex Lam S.L. [email protected] wrote:

@kangax in terms of expected behaviour we are on the same page.

You might already know, but just in case I'd like to clarify that the bug is not just comment being removed. It is this:

{{if}}blah{{/if}} getting turned into this:

{{comment}}blah{{if}} due to the assumption of sequential replacement.

I have an idea to fix this (the suggested fix above doesn't give our expected outcome...) - will post as soon as I'm back to a real keyboard.

— Reply to this email directly or view it on GitHub.

좋은 웹페이지 즐겨찾기