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

Rating Blocks not counting ratings. #35

Closed
leal-k opened this issue Oct 1, 2024 · 4 comments
Closed

Rating Blocks not counting ratings. #35

leal-k opened this issue Oct 1, 2024 · 4 comments
Assignees

Comments

@leal-k
Copy link

leal-k commented Oct 1, 2024

Concrete v9.3.4
PHP v8.3
Macareux Ratings v.1.0.1

When a rating block is put on the website, and you click it, the rating number does not go up, but it stays static at 0.
The table btC5jRatings is updated with all the Rating Buttons that are put on the website, same with the table btC5jRatingPageList, it is updated with all the Rating Page Lists you put on the website.
But the table C5jRatings does not register anything, it stays empty even though the rating buttons and rating page lists are clicked.

@biplobice
Copy link
Member

@SashaMcr
Could you please confirm this issue? I think you had resolved it previously.

@SashaMcr
Copy link
Collaborator

SashaMcr commented Oct 3, 2024

@biplobice

it works as expected

@leal-k
Is it problem on some project or on clear project?

@leal-k
Copy link
Author

leal-k commented Oct 3, 2024

I just replied to Biplob on the other conversation, on fix #34 , explaining the whole bug and fixes, but basically I was getting this error when I fixed the array to string conversion. When I updated the SQL query both bugs disappeared, and the counts were being stored, as I commented on fix #36 .
The problems may be due to the version of the ratings I was running. When I picked the issue #33 I installed the same as this person had, and started fixing from there.

@biplobice
Copy link
Member

@leal-k
Cc: @SashaMcr
If this issue isn't valid anymore, please feel free to close the issue and the PR.

@leal-k leal-k closed this as completed Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants