epam / ketcher

Web-based molecule sketcher
https://lifescience.opensource.epam.com/ketcher/demo.html
Apache License 2.0
472 stars 164 forks source link

Macro: System allow to connect block to already occupied connection (that cases invalid saved files) #3787

Closed AlexeyGirin closed 9 months ago

AlexeyGirin commented 9 months ago

Steps to Reproduce

  1. Switch to Macromolecules view
  2. Load this file: repro3.zip
  3. Establish connection from "(R2) - Right only" to (R1, R2,R3) using center-to-connection(R2) way R1 connection of (R1, R2,R3) remain unoccupied - this is ok.
  4. Establish connection from "(R1) - Left only" to (R1, R2,R3) using center-to-center way

https://github.com/epam/ketcher/assets/26869421/ed49e671-2901-452b-a999-8f8c8f462b4f

Actual behavior R1 connection remain unoccupied. Since that moment if you export canvas to ket file - it will be invalid image

Expected behavior System establishes R1-R1 connection between "(R2) - Left only" and (R1, R2,R3)

Desktop (please complete the following information):

Ketcher version [e.g. v2.4.2]. image

Documentation https://kb.epam.com/display/EPMLSOP/Types+of+connections

AlexeyGirin commented 9 months ago

Fixed.