#36 n-weekly validation

Minor
closed
nobody
MRBS (194)
5
2012-07-20
2001-11-15
Thomas Zajic
No

Selecting "Repeat type: n-weekly" but leaving the "Number of weeks" field empty makes MRBS hang for $max_execution_time seconds (as set in php.ini), and then report a fatal server error.

The enclosed patch adds an appropriate (I hope :-) check to the validate_and_submit() function in edit_entry.php, which fixes (I hope :-) this problem.

Thomas

Discussion

  • Thomas Zajic
    Thomas Zajic
    2001-11-21

    Logged In: YES
    user_id=150806

    A small update to my patch, fixes a JavaScript error when editing an existing repeating event which is not repeating n-weekly (this occured when editing a single non-n-weekly repeating event out of a series).

     
  • Thomas Zajic
    Thomas Zajic
    2001-11-21

    Logged In: YES
    user_id=150806

    Dammit ... am I the only one having trouble uploading a file
    to SourceForge with Opera/Linux? 8-/ Okay, time to fire up
    Netscape ...

     
  • Thomas Zajic
    Thomas Zajic
    2001-11-21

    Logged In: YES
    user_id=150806

    A small update to my patch, fixes a JavaScript error when editing an existing repeating event which is not repeating n-weekly (this occured when editing a single non-n-weekly repeating event out of a series).

     
  • Daniel
    Daniel
    2002-05-30

    Logged In: YES
    user_id=29285

    thanks very much!

    have applied this to CVS

     


Anonymous


Cancel   Add attachments