Re: Problems with CGI script behaviour

My advice: throw CGI.pm's stupid HTML wrappers away and write HTML directly. They are not worth it. There's no black box in front of you if you bother to write the forms directly. The only thing CGI.pm is useful for is accessing HTTP headers and query params. [1] Anyway, I'll let you on a secret. The #1 debugging tool in web programming. Ctrl-U (View Source).

Then I thought it might be a good point to use radio buttons instead. And when I add those lines of code, then I was able to click only one radio button per column and not per row as I expected (as all td's are mapped within Tr) ?!!? What am I missing here ?

Radio buttons are grouped based on their name attribute.

Another question is the behaviour of the checkbox group (named GRU in my case) where I would like "Ej" box to be unchecked if any(or more) of the previous boxes are checked - could I use onCheck here and how ? Or even better, is it possible to "make" checkboxes send a name even if they don't have a value (undef) ?

If you want only one of them to be checked, use radio buttons. oncheck is javascript and is dependent on the client browser to work. (Bad idea in my opinion, but seeing the current trends in web development, I'm in the minority.)

Checkboxes (or radio buttons) aren't submitted with the form if none are selected. You can default your params like this: $q->param(GRU => "Ej") unless $q->param('GRU');

I was trying to add a row to this table by clicking button "Addera rad". My idea was to add en empty row to a file I'm reading the table from and then call subroutine again. And it works so long that "empty" csv row is created in a file. But then, when I recall this subroutine, I get only one (first) row of a file. What am I missing here ?

Du måste addera din rad to the loop. Your loop is dependent on the file handle TEMP currently. It won't generate a single row after that. I'm afraid I'm not sure how to help there since the code looks pretty much a mess thanks to CGI.pm. (There's a map where usually a for loop would be.)

[1] Okay, I'll give it one more. It's okay for doing form prefills.

Comment on Re: Problems with CGI script behaviour Select or Download Code

Replies are listed 'Best First'.
Re^2: Problems with CGI script behaviour by Anonymous Monk on Aug 01, 2012 at 17:19 UTC
Your loop is dependent on the file handle TEMP currently. It won't generate a single row after that. I'm afraid I'm not sure how to help there since the code looks pretty much a mess thanks to CGI.pm. (There's a map where usually a for loop would be.) I suppose something as simple as changing the map from `map { ... } <TEMP>` to `map { ... } <TEMP>, ""` (where `""` is a dummy CSV row) would work but then it would only add one row and if the user wants to add multiple you would have to keep count on how many are needed and `$ENV{DEITY}`, I hate doing user interfaces with HTML forms. (But I hear HTML::FormFu might be pleasant to use.)	[reply] [d/l] [select]
Re^2: Problems with CGI script behaviour by SerZKO (Beadle) on Aug 01, 2012 at 18:06 UTC
Hej and thanks for your reply. I'll try adding a row in a loop, although my only problem with this is rewriting the table (read the changed file again gives me back one more row, but with data from only first row). But when you say that radio buttons are grouped by their name, shouldn't they be that per table row and not column? I was hoping to get following result when I press "Spara" button : `UID= UID= UID=CAME01 FNA= FNA= FNA=Carin ENA= ENA=S%E4k ENA=Plan PRE=%280157-%29 PRE=%280157-%29 PRE= BEF= BEF= BEF=0150-75385 NYTT=69090 NYTT=69099 NYTT= ADRE=Norra%20Kungsgatan%204B ADRE=Norra%20Kungsgatan%204B ADRE=Stortorget%201 GRU=P GRU=Ej GRU=Ej DOL=Nej DOL=Nej DOL=Nej CMG=Ja CMG=Ja CMG=Ja` [download] and instead I get only one DOL and CMG entry: `UID= UID= UID=CAME01 FNA= FNA= FNA=Carin ENA= ENA=S%E4k ENA=Plan PRE=%280157-%29 PRE=%280157-%29 PRE= BEF= BEF= BEF=0150-75385 NYTT=69090 NYTT=69099 NYTT= ADRE=Norra%20Kungsgatan%204B ADRE=Norra%20Kungsgatan%204B ADRE=Stortorget%201 GRU=P GRU=Ej GRU=Ej DOL=Nej CMG=Ja` [download] Any suggestion ? P.S. Thanks for the great tip with unless. I should have come up to that myself...D.S.	[reply] [d/l] [select]
Re^3: Problems with CGI script behaviour by Anonymous Monk on Aug 02, 2012 at 09:53 UTC
They are grouped by name. They don't care one bit how the table is laid out beneath them, all the care about is the `<input name="something">`. Really, check the HTML source to debug. But, I think your problem might be that you are having multiple boxes set as checked when generating the HTML. `...row1 <td><label><input type="checkbox" name="DOL" value="Ja" />Ja</label> +<label><input type="checkbox" name="DOL" value="Nej" checked="checked +" />Nej</label></td> ...row2 <td><label><input type="checkbox" name="DOL" value="Ja" />Ja</label> +<label><input type="checkbox" name="DOL" value="Nej" checked="checked +" />Nej</label></td>` [download] So the `name` attribute matches but there are multiple checked. The browsers are required to pass each and every checked checkbox back. Did you want to use radio buttons or check buttons? (Even with radio buttons you might have these kinds of problems if the HTML you supply has multiple checked.) I'm not really sure what you want since the code currently is fairly complex. You would benefit much if you made a new SOPW post with the bare minimum of code relevant to your problems, preferably one at a time. That, or create a HTML page with mere radio buttons / checkboxes and muck around it until it works. Then try to integrate that with your existing code. BTW, having two submit buttons with the same name (`spara`) doing different things (saving and adding rows) is likely to lead to problems later on.	[reply] [d/l] [select]
Re^4: Problems with CGI script behaviour by SerZKO (Beadle) on Aug 02, 2012 at 10:17 UTC
Thanks man. The reason I have multiple values checked for a GRU name is that I read the result in a hash with multiple values, and that's exactly what I want. However, I don't want it for CMG and DOL names, but there radio buttons does not work. And I'll listen to your advice to post a minimum in a new SOPW post. Cheers !	[reply]