Suggested changes to plugin_sofar_old.py

[SteveEggo]

I was having trouble displaying the Total Generation Time, getting unknown entity errors. When I put in this fix the problem was solved.
The plugin_sofar_old.py file has a duplicate names for generation_time_today, according to the spec that I have these are the register mapings aroung 0x242 to 0x245
register Function
0x242 Communication board internal info
0x243 Total Generation Time Minutes
0x244..0x245 Total Generation Time Hours

I suggest that the section of the plugin file becomes as follows:

SofarOldModbusSensorEntityDescription(
    name = "Total Generation Time Mins", #SE renamed
    key = "total_generation_time_mins", #SE renamed
    native_unit_of_measurement = UnitOfTime.MINUTES,
    register = 0x243, #SE changed from 242 to 243
    allowedtypes = AC | HYBRID,
),
SofarOldModbusSensorEntityDescription(
    name = "Total Generation Time  Hours", #SE renamed
    key = "total_generation_time_hours", #SE renamed
    native_unit_of_measurement = UnitOfTime.HOURS,
    register = 0x244,
    unit = REGISTER_U32,
    allowedtypes = AC | HYBRID,

If Wills106 agrees with these changes will he please include them in the next release.