Ember.js: 3.8.0-beta.1에서 마침표가 포함 된 키의 각 입력 중단

@jasonmit 테스트 실패로 PR을 보낼 수 있습니까?

우리는 확실히 이것을 근본 원인으로해야하지만, 이것이 결국 버그로 간주 될 것이라고 확신하지 않습니다. 특히 get(obj, 'hello.world') 는 obj.hello.world 아닌 obj['hello.world'] obj.hello.world 찾습니다.

네, 그것이 이유라고 확신합니다. each-in은 get을 사용하므로 이것은 get with dot path 티켓의 종류입니다. Mayyyybe이 특정 경우에 ES5 게터를 착륙시킨 후 여기에 들어갈 필요가 없습니다. 어쨌든 프록시와 함께 작동하지 않기 때문입니까?

특히 https://github.com/emberjs/ember.js/blob/master/packages/%40ember/-internals/glimmer/lib/utils/iterable.ts#L128

나는 이것이 버그로 간주되는지 그리고 지금 이것을 고치는 것이 다른 곳에서 문제를 일으키거나 길을 따라 문제를 일으키는 지 확실하지 않다는 것에 동의합니다.

Ya, 그리고 저는 실제로 여기서 반복되는 객체에 대한 휴리스틱 스를 기반으로 다른 일을한다고 생각합니다.

https://github.com/emberjs/ember.js/blob/7df81ec411b2da73086f63bf6a26b113dfa5a1a2/packages/%40ember/-internals/glimmer/lib/utils/iterable.ts#L290 -L298

이러한 반복기 중 일부는 get ( ObjectIterator.fromIndexable )를 사용하고 일부는 사용하지 않습니다 ( ObjectIterator.fromForEachable , MapLikeNativeIterator.from ).

이 경우 fromIndexable 를보고 있습니다. get을 사용하지 않는 유일한 문제는 cps가있는 pojos라고 생각합니다.

@chancancode-네 , @jasonmit 이 개체를 Symbol.iterable 구현하도록하여 문제를 해결할 수 있기 때문에 지적합니다.

@jasonmit 테스트 실패로 PR을 보낼 수 있습니까?

확실히, 지금 그렇게하고 문제를 연결하겠습니다.

나중에 @jasonmit 이 객체가 Symbol.iterable을 구현하도록하여 문제를

좋은 생각입니다. 해결 방법으로 시도해 보겠습니다.

키에 마침표가있는 이유가 궁금한 경우 반복되는 객체의 키는 ISO 문자열 (예 : 2011-10-05T14:48:00.000Z )이고 UTC 오프셋 이전에 마침표가 있습니다.

이것의 상태가 무엇인지 궁금하십니까? 모두 감사합니다

@ amk221 나는 이것이 해결되었다고 생각하지 않습니다. 우리는 결국 키의 기간을 피하기 위해 앱을 리팩토링했지만 분명히 모든 사람에게 이상적인 솔루션은 아닙니다.

안녕하세요 여러분, 이번에 PR을 만들어 리뷰를 요청했습니다 : D

이 문제를 해결하기 위해 할 수있는 일이 있습니까? 이게 정말 필요해 🤞

@ amk221 도우미를 추가 할 수 있습니다. 그것이 내가 문제를 해결 한 방법입니다.
그러나 이것이 중요하다는 데 동의하십시오.

이것에 몇 시간을 낭비했습니다. 저는 Ember를 처음 접했고 이것은 완전히 놀랍습니다.

내 앱의 경우 여러 호스트의 상태를 추적하므로 IP 또는 도메인 이름으로 해시 키를 지정합니다. localhost 제외한 모든 항목에 대해 정의되지 않은 결과를 반환했고이 문제를 알아 내려고 미쳐 버렸습니다.

@ devop911 은 당신의 도우미를 공유합니다 😁

@ amk221 당신이 요구하는 것은 아니지만 이것을 사용하여 결국 :

import { helper } from '@ember/component/helper';
import _ from 'lodash';

export function hashToArray([h]) {
  return _.map(h, function(value, key) {
    return {
      key: key,
      value: value,
    }
  });
}

export default helper(hashToArray);

나는이 문제도 만났다. 업데이트가 있습니까?

get 대한 일반적인 사용 사례를 회귀하지 않고이 문제를 해결할 수 있을지 모르겠습니다. 회귀가 나쁘지 않다는 것을 보여주는 벤치 마크를 얻을 수 있다면 이것을 get 에 추가하는 것이 each-in 직접 빌드해야합니다. 나는 그것이 괜찮은 회귀라고 생각합니다.

프록시 인 경우 수동으로 프록시를 확인하고 객체에 unknownProperty 를 사용할 수 있으며, 그렇지 않으면 예상대로 작동 할 수있는 each-in 사용하여 객체를 반복 할 때 특별히 일반 get 구문을 사용할 수 있습니다.

장기적으로 추적 된 소품은 곧 올 수 없습니다 😩

짧은 이야기 : Ember가 아닌 다른 것으로 전환하십시오. 모든 문제를 해결할 가치가 없습니다.

@ devop911 댓글은 매우 건설적이거나 도움이되지 않습니다. 우리는이 문제를 절대적으로 해결하고 싶지만 상대적으로 드문 엣지 케이스에 대한 사용자 애플리케이션의 성능을 회귀시키고 싶지 않습니다. 관리자로있는 것은 어려운 상황입니다.

우리는이 문제에 대한 수정을 제때 알아낼 것이며 일반적인 문제보다 약간 더 많은 작업이 필요하며 Octane의 배송 기능에 중점을 두었습니다. 나는 동의하지만 이것은 중요합니다.

헉
사용자가 키 이름을 지정하는 방법을 결정하게하세요.

@pzuraq each-in 의 키에 마침표가 포함되어 있으면 최소한 경고가 발생해야한다고 생각합니다. 회귀를 도입하지 않는 훌륭한 솔루션은 없습니다. 맞습니다.

@ devop911 엣지 케이스에 대한 저의 평가는이 문제가 열린 이후 전반적으로이 버그에 대한보고가 거의 없다는 사실을 기반으로했습니다. 이것이 모든 Ember 응용 프로그램이 의존하는 동작이라면 절대적으로 더 높은 우선 순위 수정이되었을 것입니다. 그러나 많은 사람들이 키에 마침표를 사용하지 않는 것 같습니다.

반대로 우리는 성능이 저하 될 때마다 많은 불만을받습니다. 이것이 핫 경로가 아니었다면 수정은 확실히 훨씬 더 일찍 왔을 것입니다. 내가 말했듯이, 관리자로있는 것은 어려운 상황입니다. 특히 당신이 일을 끝내려고 할 때 😩 나중에 누군가가이 일을하도록하기 위해 우리가 무엇을 할 수 있는지 알아 보겠습니다

@ rwwagner90 나는 우리가 행동을 고칠 수 있다고 생각하므로 경고를 시작하지 않고 싶지만 고칠 수 없다면 사용자에게 확실히 경고해야한다는 데 동의합니다.

https://github.com/emberjs/ember.js/pull/18296에 의해 수정되었습니다.

해결해 주셔서 감사합니다 ❤️