Hi, I tried this module with a new clean install but when I view the created meeting, I receive an error :
warning: Parameter 1 to meetings_attendance_access() expected to be a reference, value given in /includes/menu.inc on line 452.

All my modules are the stable version except for features (6.x-1.0-beta11), meetings (6.x-1.x-dev), messaging (6.x-4.0-beta7), notifications (6.x-4.0-beta7) and notification team (6.x-2.1-beta8).

Any idea ? In any case, good work !

Comments

Anonymous’s picture

Anonymous (not verified) commented
Title: Meetings module trouble » Erro when creating meeting
Component: User interface » Code

Hey. Thanks for the bug report--you're the first :).

Could you try installing the -dev version (should be available on the main meetings page) and let me know if you get the same error.

Thanks for trying out the module!

Anonymous’s picture

Anonymous (not verified) commented
Title: Erro when creating meeting » Error when creating meeting - meetings_attendance_access()
mhemart’s picture

mhemart commented

Thanks for your answer. I used your update but I still have the same error (as anonymous and admin !). I used update.php and I delete cached data for security. I was expecting a module like yours to use as a managing tool for climbing appointments.

Anonymous’s picture

Anonymous (not verified) commented

It's a very new module and fairly unstable at this point. But hopefully with more testers like yourself we can get all the bugs worked out.

I committed new code to the repo. But it could be up to 12 hours before the code "zipped" up into the -dev version that is easy to download. So if you want to download the new meetings.module file and paste it on top of the meetings.module file you have installed, the error should go away.

Please let me know if you notice other errors. I have had reports of the permissions not working as expected.

mhemart’s picture

mhemart commented

Good news : I now have a new tab called attendance but when I click on it, I go on a blank page. There is some change concerning the edit tab : I have a new error : warning: Parameter 1 to meetings_attendance_page() expected to be a reference, value given in /includes/menu.inc on line 348.
Sorry for just being a tester and not a debugger.... ;-)

I forgot to tell you that my php version is 5.3.3. I know there is some change.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Postponed (maintainer needs more info)

@mhemart - Can you please install the latest version of the code? meetings 1.0-alpha3

Please make sure you completely remove the old code. And clear your cache. Then try out the same scenarios you had before and see if you get the errors.

If you do get an error, please walk me through exactly the steps you took so I can replicate it.

Thanks!

mhemart’s picture

mhemart commented
Version: 6.x-1.x-dev » 6.x-1.0-alpha3

Hello.
I installed the last version meetings 1.0-alpha3 on a clean new install (basic parameters). I just activated your module (and required ones of course). I installed it in /site/all/modules. I created a new meeting, no problem at all but when I click on the Attendance tab, I still have a blank page. Good news, the warning message from Edit tab disappeared.

Anonymous’s picture

Anonymous (not verified) commented

When you visit the attendance tab, are you logged in as the administrator or a normal user?

When you visit the attendance tab, have you used the "Invitations and notifications" field on the original node edit page to invite any

prestondocks’s picture

prestondocks commented

HI, not sure why this thread ended here, but I have just installed your module and have the same issue. when logged in as Admin or user I have a blank page when I click the attendance tab. Did you get any further with a fix for this?

Thanks

Anonymous’s picture

Anonymous (not verified) commented

I'm very open to a patch on this if anyone has the time/interest.

Unfortunately, I'm committed to classes until I graduate in December.

litelus’s picture

litelus commented

Hello.
I've looked a bit at the code and the problem seems to be at the reference of the node passed.
Look in sites/all/modules/meetings/meetings.module on line 167 and modify

function meetings_attendance_page(&$node) {

with

function meetings_attendance_page($node) {

to make the page work.

The node doesn't seem to be changed inside the function so this shouldn't create new bugs, thou I'm not 100% sure.

That fixes the attendance page, and you can now set who attended or not (checked only with admin user).

Anonymous’s picture

Anonymous (not verified) commented

If someone can review and test this fix, I will commit it.

Thanks!

sarath.verve’s picture

sarath.verve commented

Thank you litelus ...! I had the same blank screen problem.