search

RosenborgSupporterSoftware / RUSK

RBKweb Ultimate Survival Kit
MIT License
1 stars 2 forks source link

Forbedringsforslag: Dynamisk loading av properties på (Thread|Forum|Post)Info #100

Closed havremunken closed 5 years ago

havremunken commented 5 years ago

Hva er det du foreslår å løse? Når PageContext via de forskjellige RUSKPage-implementasjonene henter inn (Forum|Thread|Post)Info-objekter, leses alle properties fra DOM umiddelbart. En god del av disse blir sjelden eller aldri brukt. Ettersom det er forsvinnende liten ytelsesstraff for å gjøre dette når innholdet blir etterspurt, gir det mer mening å gjøre dette dynamisk.

Beskriv ditt løsningsforslag Gjør om til properties med private backing-variabler. Når get() kalles, sjekk om vi har noe fra før, og hvis ikke, les fra DOM.

Dette gjøres i dag i PageContext-objektet, der public properties kun hentes inn ved behov.

havremunken commented 5 years ago

Har gjort dette for PostInfo nå.

havremunken commented 5 years ago

Det gjøres nå også for ThreadInfo og ForumInfo-objektene.