Comments (13)
@gfemorris @lukmzig I created a new issue: #17019
from pimcore.
@gfemorris I created a PR #16970
from pimcore.
@blankse thx. That should fix the search case. The issue with the object folder is still open though. Should i create a new issue for that so we can close this one?
from pimcore.
@gfemorris I can't reproduce this. In my setup the timestamps are set:
How can this values null?
from pimcore.
@blankse The browser sends null if there is no userModification set
And then it crashes
from pimcore.
@gfemorris Thank you.
I changed the 4 attributes in the db to null. Than I can't save the folder. I added a commit to the PR to fix this.
from pimcore.
Hi @blankse @gfemorris
just for my understanding, can it actually happen that all 4 values
- userModification
- userOwner
- creationDate
- modificationDate
are null?
Because in your example you are mentioning only userModification. Thank you.
from pimcore.
The db columns are nullable. So it is possible. Maybe not via admin interface. But via php script.
from pimcore.
@blankse if we change the setters to accept the null values then yes, these fields will be null-able via script. However, then this means we have empty data in the DB.
I will discuss with my colleagues this topic, for me the better solution would be to either assign system user when this happens and introduce some mechanism to assign system user on deleting of existing users.
from pimcore.
@lukmzig The null value could be exists from a earlier pimcore version without this type hints (Pimcore < 11) or someone set it directly in the db table via sql.
So if you decide to make it not nullable, the db schema should be changed and the data should be migrated.
I think there should be a difference between system user and deleted user. The system user is for changes of pimcore itself (via maintenance cronjob for example). For the created and modified dates I don't know a reason to be empty. So there we could change the db shema to not nullable and default to current time when empty it is a problem for you.
from pimcore.
@blankse I agree with you, the DB schema should not change here. This issue with the folder is bit strange and we will need to have a deeper look
@gfemorris can you please create a separate issue for the folder this? Thank you very much!
from pimcore.
The SimpleBackenSearchBundle part was fixed by #16970
The folder part will be solved in separate issue
from pimcore.
@lukmzig agreed. It's better to have a separate issue for this. Thx @blankse for creating the issue.
I am not really sure why we have folders with null values in userModification. My guess was that the client deleted the user that was in there so it went to null. Maybe that happened in an older version. Nonetheless it makes sense to fix that so that people will not have problems if they migrate to pimcore11.
from pimcore.
Related Issues (20)
- Custom View not saving the path using the Perspective Editor HOT 1
- [Improvement]: Slow loading in mapping tap in data-importer when have a lot of attributes mapped HOT 1
- [Improvement]: Add admin async loading to reverseObjectRelation HOT 1
- [Bug]: object cache is not getting generated automatically HOT 1
- Imagick Alternative for Image Transformations / Thumbnails HOT 7
- [Bug]: published field show incorect value when preview a version created in pimcore 10
- [Improvement]: Refine `phpoffice/phpspreadsheet` requirement
- [Bug]: Pimcore workflow permissions which we give in yaml file is conflicting with keybindings HOT 1
- Upgrade to doctrine V4
- [Bug]: CleanupBrickTask does not handle inial lower case letter in bricks and fieldcollections
- [Feature]: allow forbidding uploading duplicate assets
- [Improvement]: further introduce JSON-types for non-primitive data
- [Bug]: `MyDataObject::getByField($value, limit: 1)` doesn't return a single object (when named parameters are used) HOT 3
- [Bug]: Custom view broke tree displaying
- [Improvement]: Change condition in query for custom view children in tree structure HOT 1
- PhpStan Version Upgrade June
- [Improvement]: Do not warn if ffmpeg is not installed
- [Bug]: Dynamic Text Block breaks Grid options
- [Bug]: checkAllowedFormats throws an error when the format ORIGINAL is used HOT 1
- [Improvement]: In the video lectures of Pimcore Academy, the audio is muted/missing for the last 40 seconds.
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 pimcore.