Platform_Mirocod

12118 Commits

14 Branches

154 Tags

505 MiB

Tree: cd0928f0e8

Commit Graph

Author	SHA1	Message	Date
wxiaoguang	f0ba87fda8	Avatar refactor, move avatar code from `models` to `models.avatars`, remove duplicated code (#17123 ) Why this refactor The goal is to move most files from `models` package to `models.xxx` package. Many models depend on avatar model, so just move this first. And the existing logic is not clear, there are too many function like `AvatarLink`, `RelAvatarLink`, `SizedRelAvatarLink`, `SizedAvatarLink`, `MakeFinalAvatarURL`, `HashedAvatarLink`, etc. This refactor make everything clear: * user.AvatarLink() * user.AvatarLinkWithSize(size) * avatars.GenerateEmailAvatarFastLink(email, size) * avatars.GenerateEmailAvatarFinalLink(email, size) And many duplicated code are deleted in route handler, the handler and the model share the same avatar logic now.	4 years ago

Author

SHA1

Message

Date

wxiaoguang

f0ba87fda8

Avatar refactor, move avatar code from `models` to `models.avatars`, remove duplicated code (#17123 )

Why this refactor

The goal is to move most files from `models` package to `models.xxx` package. Many models depend on avatar model, so just move this first.

And the existing logic is not clear, there are too many function like `AvatarLink`, `RelAvatarLink`, `SizedRelAvatarLink`, `SizedAvatarLink`, `MakeFinalAvatarURL`, `HashedAvatarLink`, etc. This refactor make everything clear:

* user.AvatarLink()
* user.AvatarLinkWithSize(size)
* avatars.GenerateEmailAvatarFastLink(email, size)
* avatars.GenerateEmailAvatarFinalLink(email, size)

And many duplicated code are deleted in route handler, the handler and the model share the same avatar logic now.

4 years ago

1 Commits (cd0928f0e8cdfada471507de4e2921f92d716cd6)