Comments (2)
I remember wrestling with the same decision. You need some way to track all the information in the entity https://github.com/zikula-modules/DizkusModule/blob/master/Entity/ForumUserEntity.php So... you either use a related entity or user attributes which is essentially the same thing (a related entity). So - it seems like a lot of work for not much benefit
from dizkusmodule.
There is as well "proxy" class/service (I do not know if proxy is a good word for it) https://github.com/zikula-modules/DizkusModule/blob/master/Manager/ForumUserManager.php This class/service is used in most templates and I tried to use it everywhere it is intended to make entity more "smart" - it exists between controller/view and doctrine so controller can be thin it frees doctrine entities from using services and templates from repetitive logic. If implemented properly everywhere it might reduce the amount of work.
Another thing is that some pieces of information are stored in forum users entity and some like signature/avatar in user attributes which is another reason to remove forum users table completely.
On the other hand, back in the day when I was working on that part, I remember thinking that ForumUserEntity helps with something but now I have no idea what it was...hmm most probably relations https://github.com/zikula-modules/DizkusModule/blob/master/Entity/ForumUserEntity.php#L89
from dizkusmodule.
Related Issues (20)
- DizkusModulInstaller Upgrade 3.1 - 4.0 - dizkus_users - wrong column name HOT 1
- Finish refactoring and clean all the mess.
- Refactor Hooks
- Type error: Argument 1 passed to HOT 3
- Uncaught Error: The route "zikuladizkusmodule_ajax_editpost" does not exist HOT 1
- An exception has been thrown during the rendering of a template ("Unable to generate a URL for the named route "zikuladizkusmodule_user_index" as such route does not exist.") HOT 3
- Upgrade fails HOT 2
- How can I set RecentPosts for the Dizkus Block - I can not handle it HOT 5
- BBCode-Buttons and Avatars in Dizkus not working HOT 5
- release 4.1.0
- release 5.0.0
- [Hooks][Comments] - Topic provider
- 500 Server Error when calling the settings at Dizkus Module HOT 11
- Forum tree manager and data synchronisation HOT 3
- Add edit forum moderator users choser HOT 1
- Hooks non persistent
- Forum tree - recover function
- Topic split, move and join - problems and solutions
- Is Master Branch already good for Zikula 2.0.6? HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from dizkusmodule.