Thread: Inbox, Outbox, Reply problems

jaja, I also think it have sales value, it makes the communication system to work as it should, what were boonex thinking about when developing it :S??

I have not applied it yet since I've been very busy today, I will a little later and post results here.

yet even though I have seen them take at least one function from me almost word for word. They still wouldn't implement them until the next release which for a little while there was a decent amount of time to wait. All the while someone could log in and read/delete/spoof another members blog/mail/profile etc.

Hi again, I have applied half of the changes, but when applying the one to have the original message include in the reply it places the message above the textarea of the form, what I meant before is to have the message included inside the textarea (with some blank lines before it) so when the original sender receives the reply he/she is able to read/remember what was the message he/she originally sent to be getting that reply because the original message is actually included with the reply.

By applying the modification to have RE: in the subject line you do get the result I'm talking about.

I'll try to look if I can make it do what I'm talking about, if find the way I will post it here, if not, hope someone else to get it to work that way and post how, here.

Do you know how, lodlock??

Let me see what I can do. It can't be too tough, something like commenting out the previous reply stuff $ret's but keeping the db call. Then tossing $original_mes_arr['Text'] as a value to the main message's text box.

Something like that at least. I'll look at it in a few.

Sweet, I finally got it. It was driving me crazy until I realized that I couldn't refresh to see differences on compose.php

So here its, modify however you wish. Open up compose.php and find:

Code:

                $ret .= '<div class="composeTextareaBlock">';
                        $ret .= '<textarea name=text id=text class="messageTxtArea" rows="12" cols="93" name="text"  onKeyDown="javascript: this.form.textcounter.value = this.form.text.value.length;" onKeyUp="javascript: this.form.textcounter.value = this.form.text.value.length;"></textarea>';

Replace and modify to fit your desired message style:

Code:

                if ($original_mes_arr['Text'])
                        $ret .= '<textarea class="blogText" rows="12" cols="93" name="text"  onKeyDown="javascript: this.form.textcounter.value = this.form.text.value.length;" onKeyUp="javascript: this.form.textcounter.value = this.form.text.value.length;"><p>&nbsp;</p><p>&nbsp;</p><p>&nbsp;</p>------------PREVIOUS MESSAGE BELOW-----------<p>&nbsp;</p>'. process_line_output($original_mes_arr['Text']) .' </textarea>';
                else
                        $ret .= '<textarea class="blogText" rows="12" cols="93" name="text"  onKeyDown="javascript: this.form.textcounter.value = this.form.text.value.length;" onKeyUp="javascript: this.form.textcounter.value = this.form.text.value.length;"></textarea>';

For something that simple it took me to long. I must be losing it. Damn refresh. *walks away from computer balling fists and randomly cursing*

EDIT:
The cursing really wasn't that random.

Ok, again doing the changes and testing.

I found that now the text from the previous messages is included in the textarea when you reply, but some errors occur when you do several replies of the same message back and forth.

For example here is how appears a message when user 1 send it originally then user 2 reads it and replies and then user 1 reads it and hit reply (before sending), this is at the textarea of the compose page at the described moment (THIS IS NOT CODE, I will place it as code just for presentation and better understanding):

Code:

---------------PREVIOUS MESSAGE BELOW---------------



Content 2





& nbsp;



---------- -----PREVIOUS MESSAGE BELOW---------------

 Content 1

You can see the errors in red.

I think this have to do with this part of the edition done: process_line_output($original_mes_arr['Text'])

Here are some options I have found through the script, but I don't know if they will work or even what they do specifically, but just in case, to make the fix of this a little faste I will include them here:

process_smiles( process_text_withlinks_output($original_mes_arr['Text']) );
process_html_output($message_arr['Text'], 80);

Sorry for being so perfectionist, but this is about a very important feature I think and user should not be getting errors in the communication, since it is the idea of the whole site, the communication with others.

Thanks for any help, anything about this lodlock?

In messages_inbox.php change

Code:

$ret .= process_html_output($message_arr['Text'], 80);

into something larger such as:

Code:

$ret .= process_html_output($message_arr['Text'], 1800);

in compose.php find:

Code:

$ret .= '<textarea class="blogText" rows="12" cols="93" name="text"  onKeyDown="javascript: this.form.textcounter.value = this.form.text.value.length;" onKeyUp="javascript: this.form.textcounter.value = this.form.text.value.length;"><p>&nbsp;</p><p>&nbsp;</p><p>&nbsp;</p>------------PREVIOUS MESSAGE BELOW-----------<p>&nbsp;</p>'. process_line_output($original_mes_arr['Text']) .' </textarea>';

replace it with:

Code:

$ret .= '<textarea class="blogText" rows="12" cols="93" name="text"  onKeyDown="javascript: this.form.textcounter.value = this.form.text.value.length;" onKeyUp="javascript: this.form.textcounter.value = this.form.text.value.length;"><p>&nbsp;</p><p>&nbsp;</p><p>&nbsp;</p>------------PREVIOUS MESSAGE BELOW-----------<p>&nbsp;</p>'. process_html_output($original_mes_arr['Text'], 1800) .' </textarea>';

or any other number large enough to not cut off a random char. Worked for me.

Hi, ok, it seems lodlock post above this fixed the issue I mentioned about including original messages with replies, seems to be working correctly know.

By the way, notice that when you are viewing a specific message at the outbox the title of the content panel is wrong, it says that it is the inbox, you can fix this at messages_outbox.php

Find:

Code:

$_page['header_text'] = _t( "_INBOX_H1" );

Change to:

Code:

$_page['header_text'] = _t( "_OUTBOX_H1" );

Well, now I'm going to apply the changes for the outbox delete button and the independent delete of messages at inbox and outbox, I'll post results here.

Hmmm, I almost forget... the line edited at messages_inbox.php mentioned in above post is also found at messages_outbox.php, I also edited it at messages_outbox.php for both to match, I don't know if this might affect anything, but just for logic I am keeping both to be the same as they were the same before.

Take care.