Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • P purr-data
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 363
    • Issues 363
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 54
    • Merge requests 54
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Jonathan Wilkes
  • purr-data
  • Merge requests
  • !590

Gop redrect and scalars regression fix

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Ivica Bukvic requested to merge gop-redrect-n-scalars-regression into master Oct 09, 2020
  • Overview 0
  • Commits 4
  • Pipelines 1
  • Changes 3

IMPORTANT! Please note this is applied on top of the !587 to ensure there are no further regressions and is probably best applied after that merge request is merged.

  • It is supposed to toggle on and off depending on whether the subpatch has scalars only in it. It does this by also intelligently accounting for temporary text objects that are created and need to be filled by the scalar name before they are converted into a scalar class.

  • There is one condition the original implementation failed to account for: creating a new text_object in an empty canvas which erroneously still resulted in the assumption that the subpatch consisted of only scalars.

  • This patch fixes this special case.

  • To test: create an empty subpatch and enable GOP, inside the subpatch create an empty object which will erase GOP until you make that object something concrete (e.g. print). With this merge request applied, the GOP redrect will in this case never disappear (and it shouldn't). It will continue to disappear in situations where there are only scalars (as per original merge request that was merged a while ago).

Edited Oct 09, 2020 by Ivica Bukvic
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: gop-redrect-n-scalars-regression