Skip to content

RelpAppenderImpl.append() has conditional functionality based on a boolean passed in the constructor #47

@Laukkala

Description

@Laukkala

Description
RelpAppenderImpl.append() method might add an EventID or it might not include that information, based on a boolean that was passed in the constructor. It might get confusing as to why the EventID is sometimes present and sometimes not, even though the object is always the same. Separating this into different objects that always behave in a certain way would make it clearer, and you will never need to check how the object is configured to understand how it behaves.

Use case or motivation behind the feature request
It would be best if it was obvious to the developer that this object should always contain an EventID or it should never contain an EventId, as opposed to having a single object sometimes providing it and sometimes not, based on a boolean given at some prior point, when the object was instantiated.

Related issues

Additional context

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions