YaBB Development & Mods

(UTC)

Change Template: Choose Language:

Topic Summary - Displaying 10 post(s). Click here to show all

Posted by: Dandello
Posted on: Oct 4^th, 2017 at 1:35pm

- trying to reproduce the error.

Edited:

What where you trying to do when you got that? I've tested move post to different thread, move posts to new thread with new subject, move posts to new thread without new subject.

Edited:

Okay - I'm guessing here. In MoveSplitSplice find

Code (Perl)

    else {
         ${$newthreadid}{'views'} +=
           int( ${$curthreadid}{'views'} / @curthread * @postnum );
     }

and replace with

Code (Perl)

    else {
         ${$newthreadid}{'views'} ||= 0;
         ${$newthreadid}{'views'} +=
           int( ${$curthreadid}{'views'} / @curthread * @postnum );
     }

I'm guessing that the problem was on the left of the '+' sign.

Posted by: pyragony54
Posted on: Oct 4^th, 2017 at 4:40am

Quote:

An Error Has Occurred. Argument "" isn't numeric in addition (+) at /var/www/vhosts/yabbtest.de/cgi-bin/yabb2/Sources/MoveSplitSplice.pm line 685.

Posted by: Dandello
Posted on: Oct 3^rd, 2017 at 1:29pm

This really was on my to-do list. Thanks for reminding me. Embarrassed

Edited:

Okay, in MoveSplitSplice.pm find all 3 instances of

Code (Perl)

 my $convertstr = $message;

and all 3 instances of

Code (Perl)

 my $convertcut = 50;

and make the 'my' into 'our' .

Posted by: pyragony54
Posted on: Oct 3^rd, 2017 at 6:45am

Looks like everything will work.
No error message at the moment.

However, the field (Select which topic to move the selected Posts to.) Is much too long and does not appear to be limited. See Screener. You can not see anything.

Posted by: Dandello
Posted on: Oct 3^rd, 2017 at 2:16am

Yeah, something went wrong. The SVN version looks different due to indenting. I know Post.pm build 1910 is good.

Posted by: pyragony54
Posted on: Oct 3^rd, 2017 at 1:30am

Upsss, I think now something went wrong.

Code

                    $mailsent{$curuser} = 1;

Looks quite different in the SVN.

Code

            undef %{ $uid . $curuser };

Quote:

Untrapped Error :
Global symbol "$page" requires explicit package name at Sources/Post.pm line 2473. Global symbol "$page" requires explicit package name at Sources/Post.pm line 2482. Global symbol "%mailsent" requires explicit package name at Sources/Post.pm line 2490. Missing right curly or square bracket at Sources/Post.pm line 3331, at end of line syntax error at Sources/Post.pm line 3331, at EOF Compilation failed in require at Sources/Display.pm line 2126.

Posted by: pyragony54
Posted on: Oct 3^rd, 2017 at 1:10am

Not yet in the SVN? Okay, I have not tested it yet. Will do it the same time.

Posted by: Dandello
Posted on: Oct 1^st, 2017 at 2:04pm

Okay in Post.pm find:

Code (Perl)

                    if ($accept_permafull) {
                         $topiclink =
 qq~$perm_domain/$symlink/$permdate/$currentboard/$thisthread~;
                     }
                     sendmail(
                         $curmail,
                         "$notifysubjects{$curlang}{'136'}: $thissubject",
                         template_email(
                             $notifystrings{$curlang}{'boardnotificationemail'},
                             {
                                 'subject'  => $thissubject,
                                 'num'      => $topiclink,
                                 'start'    => $page,
                                 'tauthor'  => $thisauthor,
                                 'tmessage' => $thismessage
                             }
                         ),
                         q{},
                         $notifycharset{$curlang}{'emailcharset'}
                     );
                     $mailsent{$curuser} = 1;
                 }

Replace with:

Code (Perl)

                    if ($accept_permafull) {
                         $topiclink =
 qq~$perm_domain/$symlink/$permdate/$currentboard/$thisthread~;
                     }
                     if ( $thissubject && $topiclink && $page && $thisauthor && $thismessage ) {
                     sendmail(
                         $curmail,
                         "$notifysubjects{$curlang}{'136'}: $thissubject",
                         template_email(
                             $notifystrings{$curlang}{'boardnotificationemail'},
                             {
                                 'subject'  => $thissubject,
                                 'num'      => $topiclink,
                                 'start'    => $page,
                                 'tauthor'  => $thisauthor,
                                 'tmessage' => $thismessage
                             }
                         ),
                         q{},
                         $notifycharset{$curlang}{'emailcharset'}
                     );
                     $mailsent{$curuser} = 1;
                 }
                 }

This will stop the Mailer from running if any email info is missing.

In MoveSplitSplice.pm find

Code (Perl)

        if (   $FORM{'position'}
             && $newthread == $FORM{'old_position_thread'} )
         {

and replace with

Code (Perl)

        if (   $FORM{'position'}
             && $FORM{'old_position_thread'} && $newthread == $FORM{'old_position_thread'} )
         {

Posted by: Dandello
Posted on: Oct 1^st, 2017 at 1:41pm

MoveSplitSplice has always been a right royal pain in the behind.
The error in Mailer actually comes from Post.pm. The error in Display (I think) comes from the move data not being properly written because of previous problems.

So I'm working on it.

Posted by: pyragony54
Posted on: Oct 1^st, 2017 at 6:14am

Many error messages

Quote:

An Error Has Occurred. Use of uninitialized value $currentboard in concatenation (.) or string at Sources/Display.pm line 280.

Quote:

An Error Has Occurred. Use of uninitialized value in substitution iterator at Sources/Mailer.pm line 193.

Quote:

Untrapped Error :
Cannot open boards.txt at /var/www/vhosts/yabbtest.de/cgi-bin/yabb2/Sources/MoveSplitSplice.pm line 195.

It is not possible to combine two posts.
There was another error message, which I can not understand now,
she had clicked away too quickly and just did not appear.

I can not find a file boards.txt anywhere.

Edited:

Here is the missing error message

Quote:

An Error Has Occurred. Argument "" isn't numeric in numeric eq (==) at /var/www/vhosts/yabbtest.de/cgi-bin/yabb2/Sources/MoveSplitSplice.pm line 266.

Question:

Options:	Text	Color	Split Pie
Option 1:
Option 2:
Option 3:
Option 4:
Option 5:
Option 6:
Option 7:
Option 8:
Poll Comment:	Max 500 characters. Remaining characters:

Automatic End in:	days and minutes. Leave it blank if you don't want to set it now.
Enable Guest Voting:	Check this if you want guests to be able to vote in this poll.
Hide results:	Check this if you do not want the results of this poll shown until voting has been closed.
Multiple Choice	Check this if you want to allow users to vote for more than one option.
Time limit for deleting vote:	Time in minutes users are allowed to delete their vote (0 to disable delete).
Use legends for Pie Chart	Check this to use legends instead of labels next to each pie slice.
Pie Chart Radius:	Must be a value between 100 and 200 (default 100)

Name:
E-mail:
Your Verification Code is:	Please type the characters exactly as they appear in the image, without the first 2 and last 2 characters. The characters must be typed in the same order, and they are case-sensitive.
Verification Code:
Preview