I’m new here and I’m looking into contributing to Openfire. I found https://issues.igniterealtime.org/browse/OF-1795. Would it still be useful to implement caching to RosterItem in a similar way as caching is implemented for Roster?
What is your your position on refactoring existing code (mainly improving readability and removing duplicate code) and adding testcases? Is that welcomed?