Skip to content

Refactor CronExpressionEditor to use inherited getLocale() instead of locale field #6

@coderabbitai

Description

@coderabbitai

Description

The CronExpressionEditor class currently maintains a private locale field and provides its own locale handling. However, since it extends CustomField which inherits from Component, it already has access to the inherited getLocale() method.

Proposed Change

Remove the private locale field and base CronExpressionDescriptor operations on the inherited getLocale() method instead.

Benefits

  • Reduces code duplication
  • Leverages Vaadin's built-in locale handling mechanism
  • Maintains consistency with the framework's locale management approach

References

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Status

    Inbox (needs triage)

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions