передача атрибутов для пересчета изделия

[rnpoddor]

Понадобилось при вызове метода recalc, передать svg: true, для перерисовки эскиза.

[unpete]

Что мешает в том месте, где требуется svg: true, передать svg: true? По факту, вы изменили умолчания метода с save: false на save: true. Уверены, что хотели именно этого?

[rnpoddor]

По факту, вы изменили умолчания метода с save: false на save: true. Уверены, что хотели именно этого?

Я ничего не изменил, save: true было по умолчанию таким, я сделал возможным передачу attr из recalc в project.save_coordinates, если внимательно посмотреть код, передачи атрибутов не было.

[unpete]

save: true стало на входе. Против передачи атрибутов ничего не имею, но безопаснее это сделать так:

project.save_coordinates(Object.assign({save: true, svg: false}, attr));

[rnpoddor]

Оно и было save: true, атрибуты не поменялись! По мне так Object.assign лишний.

[rnpoddor]

По мне так Object.assign лишний.

нет не лишнее, с применением assign можно передать одно свойство, другие применяться по умолчанию.

[unpete]

Этот PR долго будет висеть незакрытый?

[rnpoddor]

Я же внес поправки в код в соответствии с обсуждениями, жду принятия.