`Bind all parameters to fields` assist readds existing fields

[FMorschel]

If I have this code (newly added newField):

class SubtypeNaming {
  const SubtypeNaming({
    this.prefix,
    this.suffix,
    this.containing,
    bool newField = false,
  });

  final String? prefix;
  final String? suffix;
  final String? containing;
}

And I use Bind all parameters to fields, this is the output:

class SubtypeNaming {
  const SubtypeNaming({
    this.prefix,
    this.suffix,
    this.containing,
    this.newField = false,
  });

  final String? prefix;
  final String? suffix;
  final String? containing;

  final String? suffix;

  final String? containing;

  final bool newField;
}

I'm not sure why, but it readds existing fields. Also, I couldn't figure out why it skipped prefix in this operation.

Would you like to work on this, @mosuem? If you don't, I'll happily do it.

[FMorschel]

Interestingly, if I had:

class SubtypeNaming {
  const SubtypeNaming({
    required this.prefix,
    required this.suffix,
    required this.containing,
    required bool onlyConcrete,
  });

  final String? prefix;
  final String? suffix;
  final String? containing;
}

It resulted in:

class SubtypeNaming {
  const SubtypeNaming({
    this.prefix,
    this.suffix,
    this.containing,
    required this.onlyConcrete,
  });

  final String? prefix;
  final String? suffix;
  final String? containing;

  final String? prefix;

  final String? suffix;

  final String? containing;

  final bool onlyConcrete;
}

So it re-added the existing required parameters with optional ones and re-added all fields this time.