JA Job Board Component 8,554 posts

[tekknoir Friend tekknoirJoin date: September 2009Posts: 14Downloads:0Uploads:0 Thanks: 2 Thanked: 1 times in 1 posts]

#328435

Hello,

Unfortunately it is not possible to access this site right now, it is still on my development server, which is inside a protected network, behind multiple firewalls. It is only accessible from my computer. I can not make it public until it is working.

Fortunately though, the problem still exists on your demo site. Try logging in with the employer/employer account and changing something on your profile, like the address or telephone number, it doesn’t save.

Is your demo site running the latest version? I’m beginning to wonder if the paid download file is out of date?

I had thought perhaps it was a browser-specific issue (like a javascript or xhtml error or such), but it doesn’t work in IE, Firefox, Chrome, Safari, Sea Monkey, or Opera, which account for the vast majority of browser usage.

[tekknoir Friend tekknoirJoin date: September 2009Posts: 14Downloads:0Uploads:0 Thanks: 2 Thanked: 1 times in 1 posts]

#328454

Ok, as my deadline is getting incredibly close, I’ve put aside some time for debugging, and managed to solve all of the problems I’ve been having so far.

Details of fixes made:

components/com_jajobboard/views/japrofiles/tmpl/jaform.php

Changed line 42 from:

<form name="adminForm" class="form-validate" action="index.php?userdetail=<?php echo $userdetail ?>" method="post" enctype="multipart/form-data">

To:

<form name="adminForm" class="form-validate" action="index.php?option=com_jajob&view=japrofiles&task=save&cid[]=<?php $myDebugVar =&JFactory::getUser() ; echo $myDebugVar->id ?>" method="post" enctype="multipart/form-data">

This fixes the problem where employers can not save their profile.

components/com_jajobboard/views/jaresume/tmpl/jaform.php

Changed line 42 from:

<form name="adminForm" class="form-validate" action="index.php?userdetail=<?php echo $userdetail ?>" method="post" enctype="multipart/form-data">

To:

<form name="adminForm" class="form-validate" action="index.php?option=com_jajobboard&view=jaresumes&task=save&user_id=<?php $myDebugVar =&JFactory::getUser() ; echo $myDebugVar->id ?>" method="post" enctype="multipart/form-data">

This solved the problem of Job-seekers being unable to save their resume changes.

It looks as though the jobs board component is expecting the form to be passed as $_GET vars, as opposed to the $_POST vars they are being passed as, at least the variables that tell it which part of the code to call. Thankfully it was an easy enough fix (I was expecting a lot more of the code to be ioncube encoded, but I did a recursive grep and found enough to point me in the right direction and get a handle on your code).

I have also deleted lines 320-327 of components/com_jajobboard/views/jaresume/tmpl/jaview.php

 
<img title="<?php echo JText::_('Delete resume'); ?>" src="<?php echo JURI::base().'components/com_jajobboard/images/icon_delete.gif' ?>" alt="<?php echo JText::_('Delete resume'); ?>" />
 
<a onclick="return confirm('<?php echo JText::_('Do you want to delete your resume')."?"; ?>');" href="javascript:submitbutton('remove','adminForm_<?php echo $this->item->id; ?>')" >

<span style="vertical-align: top !important;"><?php echo JText::_('Delete resume'); ?></span>
</a>
 

As I discovered that when a job-seeker deletes their resume, there is no way to create a new one, can’t even do it through the administrator controls. I could probably fix it by manually inserting a database row, but I’d rather just be rid of it before it can cause any trouble for my users.

To fix the problem with Apostrophes in fields causing a formatting error, I edited components/com_jajobboard/asset/rendfield.php …

Changed line 140 from:

$r = "<input $mouse_event $class type='text' name='" . $field->field_name . "' id='" . $field->field_name . "' $javascript value='" . $object->$value . "' style="width:" . $field->field_width . "px;height:" . $field->field_height . "px;" /> ";

To:

$r = "<input $mouse_event $class type='text' name='" . $field->field_name . "' id='" . $field->field_name . "' $javascript value='" . htmlspecialchars($object->$value, ENT_QUOTES) . "' style="width:" . $field->field_width . "px;height:" . $field->field_height . "px;" /> ";

Line 327 from:

$r = "<input $mouse_event $class name='" . $field->field_name . "' id='" . $field->field_name . "' value='" . $value . "' $javascript>";

To:

$r = "<input $mouse_event $class name='" . $field->field_name . "' id='" . $field->field_name . "' value='" . htmlspecialchars($value, ENT_QUOTES) . "' $javascript>";

And line 374 from:

$r = "<input $mouse_event $class type='text' name='" . $field->field_name . "' id='" . $field->field_name . "' $readonly value='" . htmlspecialchars ( $value ) . "' style="width:" . $field->field_width . "px;height:" . $field->field_height . "px;" /> ";

To:

$r = "<input $mouse_event $class type='text' name='" . $field->field_name . "' id='" . $field->field_name . "' $readonly value='" . htmlspecialchars ( $value, ENT_QUOTES ) . "' style="width:" . $field->field_width . "px;height:" . $field->field_height . "px;" /> ";

To fix the formatting problem I was having with the “Set up account” form, I edited:

components/com_jajobboard/views/jausergroup/tmpl/default.php

And deleted line 246:

</table>

I had also read in the open issues bugtraq thing that there is a desire to have the “Account type” area highlighted, otherwise users have a tendency to leave it on job-seeker when they mean to register as an employer.

So, again in components/com_jajobboard/views/jausergroup/tmpl/default.php, I edited line 147, changing it from:

<?php echo JText::_("Account type")?>:

To:

<h5><?php echo JText::_("Account type")?>: </h5>

Although you can use whatever header level or styling you prefer 😛

I also added a couple of <br/>’s separated by nbsp’s to the end of line 156, changing it from:

?>

To:

?> <br/> <br/>

Which just cleans up the formatting a little.

I still need to remove every instance of the phrase “Premium jobs”, as my employer’s jobs site is a free jobs site, but that should be a relatively simple process.

[Author]

Posts