Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BlocSerializationError when saving a BlElement with Ellipse geometry #75

Closed
labordep opened this issue Aug 25, 2023 · 2 comments
Closed

BlocSerializationError when saving a BlElement with Ellipse geometry #75

labordep opened this issue Aug 25, 2023 · 2 comments
Assignees
@labordep
Labels
bug Something isn't working

Comments

@labordep
Copy link
Member

labordep commented Aug 25, 2023 •
edited
Loading

Error when trying to save a BlElement with a circle.

1 - Open a new project in Pyramid
2 - Setup Project configuration (package, class and method - class side)
3 - Create an object from Add new element button and Square with randomized color
4 - Select the BlElement and set a Ellipse geometry with the button on the right section Properties
5 - Click on save button and see the error :

image

My config :

  • Windows 10 Pro
  • Pharo 11 64 bits stable
  • Pyramid main branch commit 31698fd
@labordep labordep changed the title BlocSerializationError when saving a BlElement with Circle geometry BlocSerializationError when saving a BlElement with Ellipse geometry Aug 25, 2023
@labordep labordep added the bug Something isn't working label Aug 25, 2023
@labordep labordep assigned labordep and unassigned Nyan11 Aug 26, 2023
@labordep
Copy link
Member Author

Problem came from PyramidSelectionWidgetBorderBuilder>>#topLabelFor:

@labordep
Copy link
Member Author

The problem is due to the graphical selection element, the overlay is linked to the serialized BlElement. I will send a fix.

labordep added a commit that referenced this issue Aug 26, 2023
@labordep
Add a popup for proposing debug when an execption occurs during savin…
89ff62a 
…g (for PyramidException or SerializeException) for preparing #75
@labordep labordep mentioned this issue Aug 26, 2023
Closed
labordep added a commit that referenced this issue Aug 26, 2023
@labordep
Merge pull request #76 from OpenSmock/dev
8f1cfac 
Fix #75 from Bloc-Serialization fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@labordep labordep

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Nyan11 @labordep