OpenSlides / OpenSlides/openslides-client

Amendment to motion breaks motion renderer

Open
#5,452 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
TypeScript
Stars
35
Forks
52
Avg merge
1d 9h
Merged PRs (30d)
52

Description

Describe the bug
When you create a motion with the text shown below in any text color other than default and then create an amendment with the second text shown below in the same text color (not default color), the renderer breaks trying to display the amendment. It throws the error "TypeError: Node.appendChild: Argument 1 is not an object."

How to Reproduce

  1. Create a motion with the following text using the html editor. It's important that it's not the default color.
Die Sprecher*innen können innerhalb einer Frist von 7 Tagen nach In-Kenntnis-Setzung durch einen formlosen Antrag bei dem*der DK-Präsident*in Widerspruch gegen einen oder mehrere der Beschlüsse einlegen, wenn sie zu der Einschätzung kommen, dass diese die Belange von Stipendiat*innen der Promotionsförderung in besonderem Maße berühren. Die Beschlüsse, gegen die Widerspruch eingelegt wird, werden vorerst für Stipendiat*innen der Promotionsförderung nicht wirksam. Beschlüsse, gegen die kein Widerspruch eingelegt wird, werden nach Ablauf der Frist für Stipendiat*innen der Promotionsförderung wirksam. Die Wirksamkeit der Beschlüsse für alle übrigen Stipendiat*innen bleibt davon unberührt.
  1. Create an amendment to this motion, changing the whole paragraph (lines 1-9), delete the whole paragraph and put in the following text, using the html editor, the color must be the same as in the motion.
Gemeinsam mit zwei weiteren Mitgliedern des Senats können diese durch einstimmiges Votum innerhalb einer Frist von sieben Tagen nach Abschluss der Delegiertenkonferenz das Aufschieben des Inkrafttretens eines oder mehrerer der Beschlüsse der Delegiertenkonferenz beschließen, wenn sie zu der zu der Einschätzung kommen, dass diese die Belange von Stipendiat*innen der Promotionsförderung in besonderem Maße berühren. Alle übrigen Beschlüsse, die nicht satzungs- oder ordnungsändernde Beschlüsse sind, treten nach Ablauf der Frist in Kraft.
  1. Navigato to amendment detail
  2. If you tick the box "Show entire motion text", the error will be shown.

Expected behavior
The changes get rendered correctly.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the motion renderer used by the amendment detail view and reproduce the failure by opening the amendment and selecting “Show entire motion text” after using the specified non-default color. Trace the Node.appendChild error, then verify that the amended motion renders correctly with matching text colors and that the reproduction no longer throws.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.