Re: The JobOffer.php is not "HTML 4.01 Transitional" valid

[Davi Leal]

MJ Ray wrote:
> Davi Leal wrote:
> > The "Job_Offer.php" page has some errors, which I think we will
> > not able to fix?.  If curious read its CVS log. Therefore, I will
> > close this task: https://savannah.nongnu.org/task/index.php?6546
>
> I probably don't understand what that page is trying to do (it
> variously tells me to log in first or that access is not granted to
> me),

That is the Access Control List (ACL) working.

  https://www.gnuherds.org/Job_Offer.php?JobOfferId=11

Obviously, it is needed to register and log in at GNU Herds to access to the 
form which allows to fill or update a job offer at the GNU Herds webapp.  
That is to say, it is not allowed to fill an anonymous job offer.


Access Control List:

  * If you are not logged   you are not allowed to access it.

  * Due to such form is for edit the job offer, if you are
    not the owner,   you are not allowed to access it.

  * Anyway, you can access this
      https://www.gnuherds.org/Job_Offer.php
    just being logged, due to it is not for edit an old offer
    but create a new one.

Reference:
  Layer-4__DBManager_etc/DB_Manager.php
  Layer-4__DBManager_etc/Access_Control_List.php


> but why not put some <option ... />s into the select tag?  Could 
> be more accessible than whatever that is doing with javascript.

Yes, the JavaScript fills those four <select>s elements with its proper 
<option>s. The problem is that such option values are dynamically created 
while the user is working on his browser.

If we add some dumb <option> when the server send the web page to the user 
browser, just to pass the W3C checker and so be specification compliant, the 
JavaScript will have to remove it, before start to run.   It is too much 
complex. Maybe it is better leave it as it is now?. There is others tasks 
more important for me. Here all are free to do what we want.

CVS log at 
http://cvs.savannah.gnu.org/viewcvs/gnuherds-app/gnuherds-app/Layer-0__Site_entry_point/templates/Job_Offer_form.tpl?rev=1.12&view=log


P.S.: Thanks for your comments MJ. They are really really appreciated. Let me 
know if you yet disagree.  Though you disagree, I think I will not carry out 
such modification now, at least your rationale expose it be very important. 
It is due I have others tasks: I am going to add the for=".." to the labels.  
However it is necessary to know your opinion. I have took note about it in 
the Savannah Task Manager to be analyzed later. I have opened again that 
task, but with very low priority:
  Task: https://savannah.nongnu.org/task/index.php?6546

If you register at our savannah project following the below steps you will 
able to raise the level of a tasks, etc.
  http://www.gnuherds.org/GNU_Herds_Hackers_Guide.php#The_CVS_server

Regards,
Davi