MLWebTechnologies / PrayerCenter

Online Prayer component
http://mlwebtechnologies.github.io/PrayerCenter/
3 stars 6 forks source link

error when submitting request using the module... #18

Open GODpleasers opened 6 years ago

GODpleasers commented 6 years ago

error when submitting request using the module... error0

Any how i disabled it to it can be fixed thanks for looking.

myale commented 6 years ago

i have same issue... but let me extend my comment here, its not just he above module... ITS ALL OF THEM (4) it would be neat to maybe have a redirecting page as well

GODpleasers commented 6 years ago

At least we are not the only ones.

MLWebTechnologies commented 6 years ago

I'm not able to reproduce this error. My assumption is, from the screenshot, that this is a javascript error but I do not know from where. Would you have it on your website that I can see and view the java error in a web browser?

GODpleasers commented 6 years ago

we could enable it so you can try it. please let us know when you are ready and we will.

myale commented 6 years ago

Hello mike,

I had enabled all the mods I have for the prayer center I had them disabled so I don’t completely remember what the errors are,

And I want to be able to list payers in like a page format organized,

The subscription mod I don’t know if there is a way for me to see all who have subscribed and if they are receiveing the prayers listed etc…

I am still learning the component and such…

Thanks…

Mark

www.mylordsministries.com

From: Mike [mailto:notifications@github.com] Sent: Saturday, May 26, 2018 8:30 PM To: MLWebTechnologies/PrayerCenter Cc: myale; Comment Subject: Re: [MLWebTechnologies/PrayerCenter] error when submitting request using the module... (#18)

I'm not able to reproduce this error. My assumption is, from the screenshot, that this is a javascript error but I do not know from where. Would you have it on your website that I can see and view the java error in a web browser?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MLWebTechnologies/PrayerCenter/issues/18#issuecomment-392298776 , or mute the thread https://github.com/notifications/unsubscribe-auth/ADylepKgFAchu-m7vJGjpwggebC0QrgQks5t2gGCgaJpZM4T0SEM . https://github.com/notifications/beacon/ADyleryeOqYtnGL45oh8lVj6DdyU0q3fks5t2gGCgaJpZM4T0SEM.gif

GODpleasers commented 6 years ago

Hi mike,

We are the ones that submitted the image.

Any how we have enabled the submit request module on our site for you to test it.

We don't normally like thing that don't work enabled so please hurry up and do your testing.

Thank you. https://godspeaksinternetradio.com/ its under the daily bible reading.

MLWebTechnologies commented 6 years ago

Hi, Sorry I was at church earlier. Do I need to login? I'm not seeing that module enable on your site, you may have already removed it again.

GODpleasers commented 6 years ago

Hi, Sorry I was at church earlier. Do I need to login? I'm not seeing that module enable on your site, you may have already removed it again.

Sorry about that had it set for registered. Any how I changed it to public you should see it now. https://godspeaksinternetradio.com/

What churc do you go to do that have a website? We like listening to sermons online.

MLWebTechnologies commented 6 years ago

Ok, I have the error output and I will see what I can find to correct that issue. Thanks.

GODpleasers commented 6 years ago

MLWebTechnolog Ok, I have the error output and I will see what I can find to correct that issue. Thanks.

Thank you

myale commented 6 years ago

Hey there, if you need to register and login go agead,

Mark

From: Mike [mailto:notifications@github.com] Sent: Sunday, May 27, 2018 12:52 PM To: MLWebTechnologies/PrayerCenter Cc: myale; Comment Subject: Re: [MLWebTechnologies/PrayerCenter] error when submitting request using the module... (#18)

Hi, Sorry I was at church earlier. Do I need to login? I'm not seeing that module enable on your site, you may have already removed it again.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MLWebTechnologies/PrayerCenter/issues/18#issuecomment-392350720 , or mute the thread https://github.com/notifications/unsubscribe-auth/ADyletUP_z9Dn7dhoFJOnUeRzOMO6dc0ks5t2uemgaJpZM4T0SEM . https://github.com/notifications/beacon/ADylemyzyWUqyShAvKJztQ_4ysRVmH7_ks5t2uemgaJpZM4T0SEM.gif

GODpleasers commented 6 years ago

We re-enabled it so you can test it at your convenience. https://godspeaksinternetradio.com/

GODpleasers commented 6 years ago

Please, Any word on fixing this yet. https://godspeaksinternetradio.com/

We enabled it in public view so you can test it but don't like things that don't work on our site?

MLWebTechnologies commented 6 years ago

Don't have a fix yet, suspect an issue with redirects and SSL enabled site. I have what I need, please disable on your site again.

GODpleasers commented 6 years ago

MLWebTechnologies Don't have a fix yet, suspect an issue with redirects and SSL enabled site. I have what I need, please disable on your site again.

Ok thank you look forward to seeing it working.

GODpleasers commented 6 years ago

Hi any leads on when this will be fixed?

MLWebTechnologies commented 6 years ago

Ok, I believe that I may have found the issue. Depending on the supported version of PHP on your site, the PHP function "split" has been deprecated and may or may not work correctly. In PrayerCenter, there is a function "PCcheckEmail" that calls the "split" function. It has been replaced with the PHP function "explode".

If you are comfortable with editing the PHP files on your site, you can search for the function "PCcheckEmail" in the "components/prayercenter/helpers/pc_class.php". On the 7th line of that function, change "list( $username , $domaintld ) = split( "@" , $email );" to "list( $username , $domaintld ) = explode( "@" , $email );"

I can also send you the "pc_class.php" file that you can replace on your site via FTP or file explorer on your webhost's CPanel.

Please let me know if that fixes the issue. Thank you!

GODpleasers commented 6 years ago

Hi we use php Version 7.0.30 on our site.

Thanks we are not comfortable editing PHP files on our site, If you can please send the edited "pc_class.php" file uploading it would be the best way for us.

Programmer1971 commented 6 years ago

Hi On www.GodsStation.org we used your component for several years but just recently updated from 2.5 joomla to 3.8 and upgraded to the 2.04 PrayerCenter version. We too had the error with the SPLIT function and I just tried the change to EXPLODE which appears to fix that error - but now when entering in an email for subscribe, we get a new error "0 - Using $this when not in object context" which also may be due to the newer PHP here. While I do a bit of PHP programming it takes me a while to chase those types of errors so your input on this might save me some time and be helpful for others when/if they see them also. We do really like your component and it is doing some good things. By the way the EXPLODE swap for SPLIT did seem to work - I think this new error is further into the function.
Temporarily the new version of our site is at a subdomain http://new.godsstation.org while the older 2.5 joomla is at http://www.godsstation.org if you wish to look. Hosted on 1and1 and the new directory is set for 7.2 PHP, while the old is still 5.6 if memory serves.
Thanks again for your work. God Bless you.

Programmer1971 commented 6 years ago

An added comment about the newer PHP caused errors. It is only showing on the SUBSCRIBE command and apparently when it is near finishing. A persons email is saved, and subsequent attempts will show a duplicate entry. I am moving the newer version of the site to the main domain name and will be dropping the older version as we need to get the newer joomla version in the public even if this error causes a bit of confusion. I will keep looking and if I find a solution before you can get time to advise here, I will publish it so that it might help those who need it. Seems PHP 7.2 is breaking more than a few programs with its forward progress.