Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

no access when not admin #19

Open
4 tasks done
arboc opened this issue Apr 8, 2024 · 6 comments
Open
4 tasks done

no access when not admin #19

arboc opened this issue Apr 8, 2024 · 6 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request
Milestone

Comments

@arboc
Copy link

arboc commented Apr 8, 2024

Checklist before I submit this issue report

I confirm that:

My environment is:

Key Value Comments
PHP version: 7.4.33
Contao version: 4.9.42
MetaModels version: 2.2.0 2.2.1 for this extension
Installation via composer yes
Installed MetaModels packages no other

Issue description

when I am not an admin user, I cannot edit any content article elements.

Steps to reproduce

  1. Login as a ususal user
  2. Edit a MM Element with a BE Formular Element: contentarticle
  3. Try to click "Auswahl ändern"
  4. A Modal with an error occurs

Describe the behaviour of the application

the modal shows an error: "Ein Fehler ist aufgetreten!"

Describe the expected behaviour of the application

showing a list of Content Elemtns

Screenshots

image

Additional Information

In MetaModels\AttributeContentArticleBundle\Table\ArticleContentLine 324 ff:

// Invalid ID
if ($objContent->numRows < 1) {
    System::log('Invalid content element ID ' . $accessId, __METHOD__, TL_ERROR);

change objContent->numRows to $objContent->rowCount() , to find the number of rows.

arboc added a commit to arboc/attribute_contentarticle that referenced this issue Apr 8, 2024
@arboc arboc mentioned this issue Apr 8, 2024
6 tasks
@zonky2
Copy link
Contributor

zonky2 commented Apr 8, 2024

@arboc Does the bug also occur in MM 2.3?

@zonky2 zonky2 self-assigned this Apr 8, 2024
@zonky2 zonky2 added this to the 2.2.0 milestone Apr 8, 2024
@zonky2 zonky2 added the bug Something isn't working label Apr 8, 2024
@arboc
Copy link
Author

arboc commented Apr 8, 2024

I have not tested in MM 2.3.
atm we do not have installed MM 2.3. we will upgrade next month.

@zonky2
Copy link
Contributor

zonky2 commented Apr 8, 2024

pls delete your response email(s) at github ;-)

MM 2.2 is no longer being actively worked on, as Contao 4.9 has expired - if necessary, a backport to MM 2.3 could be made

@arboc
Copy link
Author

arboc commented Apr 8, 2024

offtopic:
when will MM 2.3 be available under packagist? I see MM2.2 only.

@zonky2
Copy link
Contributor

zonky2 commented Apr 8, 2024

@zonky2 zonky2 added the enhancement New feature or request label Apr 9, 2024
@zonky2
Copy link
Contributor

zonky2 commented Apr 9, 2024

Method checkAccessToElement is no longer included in MM 2.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants