Jump to content
MakeWebGames

Recommended Posts

Posted

Hey all... I coded a brand new mailbox from scratch.

The others were untidy and hard to understand the mailbox. It was to pushed up together.

Here is my mailbox very easy readable and same as the code. :wink:

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

 

Hope you like.

EDIT: Please use this updated one. It has the bugs idealed out and now showing the emoticons that can be used.

  • Replies 58
  • Created
  • Last Reply

Top Posters In This Topic

Posted

Re: New improved mailbox.

dont know if this will work but i tried

since in killah`s mod you cant preview the smileys i tried adding them my self :|

please comment anything wrong

PS

I never did any PHP so better back up any files just in case :lol:

thanks

 

 

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

Posted

Re: New improved mailbox.

 

Yes that should work but unfortunatly makes my codes look bad :roll:

ahh oppss well if someone can delete then, i didnt do on purpose :-)

but at the same time it works so :-P

Guest Anonymous
Posted

Re: New improved mailbox.

If it works, that's always a bonus, but personally I don't feel his "codes" (why do people insist on using that word...?) look good to start with.

There's obviously a misunderstanding about protecting input variables:

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

 

Incorrectly checking the 'act' element and no need to escape.

Plus mixing the display with code -- always messy, however I am guilty of that in some of the source files I have presented, so I'll forgive that one.

I've not had a chance to test it as yet, but a quick scan of the source, it looks a reasonable extension.

Posted

Re: New improved mailbox.

 

If it works, that's always a bonus, but personally I don't feel his "codes" (why do people insist on using that word...?) look good to start with.

There's obviously a misunderstanding about protecting input variables:

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

 

Incorrectly checking the 'act' element and no need to escape.

Plus mixing the display with code -- always messy, however I am guilty of that in some of the source files I have presented, so I'll forgive that one.

I've not had a chance to test it as yet, but a quick scan of the source, it looks a reasonable extension.

Its just for security that im checking the $_GET['act'].

Guest Anonymous
Posted

Re: New improved mailbox.

Misguided security - There is no need to escape that, as all you are doing with ,act, is a simple string matching in a switch statement.

KISS - aka Keep It Simple Stupid...

Minimalistic code is easier to debug, easier to understand and will run faster.

Posted

Re: New improved mailbox.

 

only problem i found with this is that it doesnt show you that you recived a msg

(1)

Yeah, when you send a mail it doesn't show that they got a mail

[Mail (1)]

Don't know what's wrong tried to fix it but still no alert when they get a message?

Posted

Re: New improved mailbox.

 

http://criminalexistence.com/ceforums/i ... c=17855.75

^works perfect just doesn't have the smiley option which i think is a little stupid anyway

 

Hey all... I coded a brand new mailbox from scratch.

are you sure about that kyle the code looks very similar to the edited version of the one on here... weird huh? anyone else think thats a little weird?

Like i said befor!! i did code this from scratch read over the coding! now after this message your going to be banned from all my posts if i can still do that.

 

This is a good re-make of Richard mail box, Nice one Killah

Funny how you say re-made of richards mail box but please read over the code befor you say stupid shit?

 

Unknown column 'new_mail' in 'field list'

You using v1? because this is scripted for v2.!

Posted

Re: New improved mailbox.

Good Mod, I did use it, But it got annoying as it wasn't showing the Mail's :-P

I tried adding the ("UPDATE users SET new_mail=new_mail+1 WHERE userid={$to}") But It's already there :lol:

Posted

Re: New improved mailbox.

 

Good Mod, I did use it, But it got annoying as it wasn't showing the Mail's :-P

I tried adding the ("UPDATE users SET new_mail=new_mail+1 WHERE userid={$to}") But It's already there :lol:

 

Find

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

 

on or around line 293 and remove the apostrophe's from either side of {$who}

Do the same on or around line 174 and remove them from {$ir['userid']} on or around line 183 and {$id} on the line after.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


×
×
  • Create New...