in reply to Object Identifier?

> doesn't work

Doesn't work!

Please tell us about the errors you get.

Your code is incomplete, the last elsif isn't properly closed.

This is definitely a syntax error

$last240_flag = "checked";"; ^^

Please have a look at SSCCE and Basic debugging checklist

> and my admin retired so I cannot get any help

And maybe think about hiring a new admin.

Cheers Rolf
(addicted to the Perl Programming Language :)
Wikisyntax for the Monastery FootballPerl is like chess, only without the dice

Replies are listed 'Best First'.
Re^2: Object Identifier?
by damfer21 (Novice) on Oct 22, 2018 at 12:15 UTC

    Thanks for the reply. I only inserted part of the code to show where the addition was among its similar elements. Every other $lastflag works. This code was written and worked for a decade. I just wanted to add the new flag. I'd hire a new admin if I had the money. I am doing what I can to keep things going at this point, hence the plea for help here. I would just like to know why my additional elseif for $last240_flag does not work. The error I get is that it selects all recipients instead of ones that have been active for last 240 days. All the other last flags perform correctly.

      You write:
      The error I get is that it selects all recipients instead of ones that have been active for last 240 days. All the other last flags perform correctly.

      I am pretty sure that there's more to this application than the script you've shown us. You seem to have added a radio button for the clast60 field with a value of 4 and a meaning of Last 240 Days in the script which offers the selection. But this is not the script doing the database query: There is no place in this code where you actually use that parameter. So I guess that you need also to make this new value known to your other script camp_copy_schedule_save.cgi. If that script has the same programming style, then a newly added value of "4" might end up in the final else branch of some if/elsif/else abomination - and treat it as if the clast60 has not been provided at all.

      Good luck! You'll need it.

        Thank you. I did exactly what you say I did, figuring I could copy one of the other 'last' flags. I'll look in camp_copy_schedule_save. The interesting thing is that I have not noticed where that definition is. I've searched hundreds of files.

        As I have said, I didn't create this code, it just runs on my client. The fact that I was able to copy the code is a miracle.