Yes yes! Thanks, Bruttosozialprodukt!joedf wrote:alright sorry about the packing thing anyway. no problemo, mi amigo!
thank Bruttosozialprodukt too!
Watching you work this out is kinda cool. Sorta like a surgical theater.
Yes yes! Thanks, Bruttosozialprodukt!joedf wrote:alright sorry about the packing thing anyway. no problemo, mi amigo!
thank Bruttosozialprodukt too!
LOLEzriilc wrote:Watching you work this out is kinda cool. Sorta like a surgical theater.
It's been a long time since I've been under the hood of phpBB, but don't most apps like it have open handles already going for db and other stuff?Bruttosozialprodukt wrote:What do you mean by "using phpBB"?
Our way is very likely more efficient then including a buch of phpBB libs etc.
We are simply using phpBB to verify the user is logged in and then we read our stuff from the database, which shold be 100% safe.
I may be misunderstanding how this executes. I apologize - I didn't get it working yet since I'm using SQLite, and I didn't read the JS at all.Bruttosozialprodukt wrote:But you realize that the php file that we call from javascript is it's own file, right?
So there are no handles or anything form phpBB by default and since php script instances can't communicate with each other I don't really see how such handles could be of any help.
Also I don't think that the mySQL functions do return a handle or something like that.
Use an interface like phpmyadmin to change the "Topic_watch table"Ezriilc wrote:Once I get this tweaked to use SQLite, how can I trigger a notification for testing?
Well, that duplicates even more code and so seems to be a step in the wrong direction to me. Also, it seems like that might be a bad idea from a security point of view. Personally, I consider cookie logins to be one of the least secure aspects of user control, and it might be wiser to trust phpBB to handle it.Bruttosozialprodukt wrote:For a future update we could maybe validate the user login cookie manually to get rid of the common lib include.
Ah yes, thanks. But I'll use phpLiteAdmin.joedf wrote:Use an interface like phpmyadmin to change the "Topic_watch table"
I do believe you. That makes a lot of sense.Bruttosozialprodukt wrote:Yeah, of course this would kind of duplicate code because there will be multiple file that can validate a login cookie, but therefore it would drastically reduce CPU/RAM usage which is much more valuable than disk space.
Just believe me loading 10000 functions/variables into the RAM just because you want to use one of those functions is not very efficient.
Users browsing this forum: No registered users and 35 guests