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.

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

Fix für NullPointerException in MitgliederStammdaten #21

Merged
merged 2 commits into from
Oct 31, 2022
Merged

Fix für NullPointerException in MitgliederStammdaten #21

merged 2 commits into from
Oct 31, 2022

Conversation

kpatzwald
Copy link

Dieser Pull Request behebt die Nullpointerexception in den Stammdaten -> Mitgliedskonto, wenn das Mitgliedskonto nicht ausgewählt wurde. Es erscheinen nun passende Fehlermeldungen.

#20

@@ -42,6 +42,10 @@ public class SpendenbescheinigungAction implements Action
@Override
public void handleAction(Object context) throws ApplicationException
{
if (context == null || !(context instanceof MitgliedskontoNode))

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dein Editor verwendet hier tabs anstatt spaces kannst du das noch abändern?
sollte zumindest innerhalb von einem file gleich bleiben.

@@ -35,6 +35,11 @@ public class MitgliedskontoIstLoesenAction implements Action
@Override
public void handleAction(Object context) throws ApplicationException
{
if (context == null || !(context instanceof MitgliedskontoNode))

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

selbe topic mit spaces/tabs

@@ -36,6 +36,11 @@ public class MitgliedskontoDetailSollLoeschenAction implements Action
@Override
public void handleAction(Object context) throws ApplicationException
{
if (context == null || !(context instanceof MitgliedskontoNode))

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

selbe topic mit spaces/tabs

Copy link

@phschoen phschoen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change the tabs to spaces.
Rest is fine :)

@phschoen phschoen merged commit e0b2c87 into openjverein:master Oct 31, 2022
heinerjost pushed a commit to jverein/jverein that referenced this pull request Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants