mapbox/mapbox-gl-js

Add an error message for calling "GeoJSONSource#setData" for a style that doesn't exist

Open

#2,602 建立於 2016年5月22日

在 GitHub 查看
 (3 留言) (0 反應) (0 負責人)JavaScript (10,532 star) (2,203 fork)batch import
feature :green_apple:good first issue

描述

Can this error message be improved?

(link removed)

Step 1) press the [+] button Step 2) Notice the specific error message

dispatcher.js:32 Uncaught TypeError: Cannot read property 'send' of undefined

The dispatcher.js:32 that doesn't give any hints whatsoever for me and probably other api users.

貢獻者指南

技術棧
javascript
領域
frontend
議題類型
feature
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
under 1 hour
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
JavaScript basicsMapbox GL JS source API
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
70
研究方向
The error occurs in dispatcher.js at line 32 when calling GeoJSONSource#setData on a style that does not exist. To add a meaningful error message, investigate the GeoJSONSource.setData method in the mapbox gl js source code. Determine where the style reference is used and add a check to throw a descriptive error if the style is missing. The issue provides clear steps to reproduce and the desired outcome is a user friendly error instead of an obscure TypeError.