Part-DB-legacy icon indicating copy to clipboard operation
Part-DB-legacy copied to clipboard

Published 20 hours ago • verified publisher icon Part-DB

Undefined index: perms_labels

Open chrisnoisel opened this issue 5 years ago • 3 comments

Hello, I've have a setup of the current master branch on apache/mariadb, database is version 26. I noticed in my log that php complains about a missing "labels" permission. I looked it up, here what happens :

the admin user, coming from a fresh partdb install, has a "perms_labels" set to 0, so every request to label permission is translated to being group-inherited ($val === BasePermission::INHERIT) Unfortunately the database table "groups" doesn't include a "perms_labels" column, thus the warning is generated.

This could be addressed by creating a new database version, but looking at update scripts, I'm unsure how to properly generate a migration. This doesn't look too critical (basically, labels permissions are disabled), is this worth fixing right now ? and if so, can you help ?

chrisnoisel avatar Dec 20 '19 22:12 chrisnoisel

I am not sure if I will fix that. When I change the DB structure I have to change the Migrations of Part-DB 1.0 too (they are currently targeting the structure of Part-DB 0.5.6). The label permissions are fixed in Part-DB 1.0 and they are not really needed (at least they are not security relevant)...

Any other opinions on this?

jbtronics avatar Jan 05 '20 20:01 jbtronics

Looking at your commit activity I figured you would rather focus one the 1.0, which is totally fine. If partdb 0.5.6 is dev-deprecated then it would be useful to have like a roadmap on the current 1.0 development (what is done, work-in-progress, etc.) or some insights about what you're going to do. I may have some features I'd like to add but right now, I've found it's harder to dig in the new version (well, not knowing Symfony doesn't help, so I've some homework to do on my own)

chrisnoisel avatar Jan 05 '20 22:01 chrisnoisel

After you have understood some basic ideas of Symfony the Part-DB 1.0 codebase should be easier to understand than the version in this repo (also the development is so much more comfortable).

I have written something about my plans for Part-DB and my ideas for a roadmap in the new repo (thanks for your suggestion): https://github.com/Part-DB/Part-DB-symfony/issues/9

jbtronics avatar Jan 06 '20 22:01 jbtronics