Right click identification

[viatkinviatkin]

Идентификация по правой кнопке. РГИС ПК 5 очередь. 5 этап

• added the ability to identify features on the map by right click
• maptool-options service created
• right-click identification should be disabled for:
  • feature edit
  • map export

[DaryaNeko]

Вот это сочетание немного спорное, т.к. логика по открыванию сайдбара и вкладкам - это больше к прикладному коду относится, а ты дорабатываешь технологию.
Есть возможность идентификацию включать и по кнопке в тулбаре, это у нас она там выключена и унесена в сайдбар.

Поэтому идентификацию (соответствующий tool) хорошо бы включать по имени, а не через вызов из прикладного кода

[viatkinviatkin]

Исправил, вызываю identify-visible-rectangle напрямую

[DaryaNeko]

а тут бы может запоминать где-то предыдущий инструмент и включать именно предыдущий, а не просто drag
может быть проверять что он и списка "нормально включаемых" ) если нет, тогда уже drag

[viatkinviatkin]

Исправлено, определено в rightClickAvailiblePrevMapTools

[DaryaNeko]

Давай поставим значение по умолчанию зачем издеваться над прикладными проектами, и заставлять гадать, если у нас пока по сути один инструмент предполагается?)
Возможность переопределения - это отлично. но вот необходимость гадать - это уже плохо.

[viatkinviatkin]

Исправил, всё тут

ember-flexberry-gis/addon/services/maptool-options.js

Lines 3 to 20 in 7617335

	export default Ember.Service.extend({
	/**
	Flag: indicates whether tool is performed when right-clicking on map.
	@property isRightClickToolAvailable
	@type Bool
	@default false
	*/
	isRightClickToolAvailable: false,
	/**
	Map tool that should be enabled when right clicking
	@property rightClickToolName
	@type String
	@default 'identify-visible-rectangle'
	*/
	rightClickToolName: 'identify-visible-rectangle',

[DaryaNeko]

И тут тоже можно значение по умолчанию задать. хотя бы тот же drag и zoom-in/zoom-out

[viatkinviatkin]

Исправил, всё тут

ember-flexberry-gis/addon/services/maptool-options.js

Lines 3 to 30 in 7617335

	export default Ember.Service.extend({
	/**
	Flag: indicates whether tool is performed when right-clicking on map.
	@property isRightClickToolAvailable
	@type Bool
	@default false
	*/
	isRightClickToolAvailable: false,
	/**
	Map tool that should be enabled when right clicking
	@property rightClickToolName
	@type String
	@default 'identify-visible-rectangle'
	*/
	rightClickToolName: 'identify-visible-rectangle',
	/**
	Right click map tool options
	@property rightClickToolProperties
	@type Object
	@default null
	*/
	rightClickToolProperties: null,

[DaryaNeko]

давай все таки тут имя зашивать не будем, иначе какой смысл от того, что мы дали его определять? пусть будет на совести прикладного разработчика

[viatkinviatkin]

Исправил

[sonarcloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud