-
AuthorPosts
-
brentwilliams2 Friend
brentwilliams2
- Join date:
- January 2013
- Posts:
- 231
- Downloads:
- 0
- Uploads:
- 23
- Thanks:
- 4
- Thanked:
- 33 times in 1 posts
November 8, 2013 at 4:17 pm #511652THANK YOU!
It is almost completely fixed – two things and we’ll be all set!
1) When you use the main filter on the site, a text field is included in that search. For example, I have a “City” field, and when using the filter, that field is included in the search. But seems to be missing in the alert. Does that make sense?
2) The fix does not work when editing and saving a job. I noticed some alerts were all messed up, and we traced it back to the fact that those jobs had been created, and then later edited. So your fix needs to be applied to the part when editing a job, as well.
By the way, it appears that you deleted the entire alerts table. Did you make a backup? I have lost a fair amount of data otherwise… (I restored a backed up copy from a week ago, but we’ve had a lot of alerts since then)
brentwilliams2 Friendbrentwilliams2
- Join date:
- January 2013
- Posts:
- 231
- Downloads:
- 0
- Uploads:
- 23
- Thanks:
- 4
- Thanked:
- 33 times in 1 posts
November 10, 2013 at 4:44 pm #511751(Update to last post: You might consider removing alerts for edited jobs. If an employer keeps editing their job, it could send out several alerts for the same job, which is not good. It would probably be best for an alert only to be sent on the original job creation.)
Luna Garden ModeratorLuna Garden
- Join date:
- July 2011
- Posts:
- 2617
- Downloads:
- 80
- Uploads:
- 96
- Thanks:
- 78
- Thanked:
- 453 times in 425 posts
November 11, 2013 at 3:44 am #511773Hello BrentWilliam,
<blockquote>1) When you use the main filter on the site, a text field is included in that search.</blockquote>
The fields in Job Alert are fixed, and do not relate to the custom field on Main Filter on the site, cause we can’t control it.
<blockquote>2) The fix does not work when editing and saving a job. </blockquote>
<blockquote>You might consider removing alerts for edited jobs. If an employer keeps editing their job, it could send out several alerts for the same job, which is not good. It would probably be best for an alert only to be sent on the original job creation.</blockquote>
Yes, It’s a bug, I have discussed with the developer, and he’ll consider to improve the processing of Job Alert. I have created an issue on our project manager for you to follow its status: http://pm.joomlart.com/browse/JAJBJ25-338
<blockquote>it appears that you deleted the entire alerts table. Did you make a backup?</blockquote>
Sorry BrentWilliam, the data in that table contains some errors, so the developer decided to delete it all. He still keeps the backup of this table, if you want, the backup file is attached.
brentwilliams2 Friendbrentwilliams2
- Join date:
- January 2013
- Posts:
- 231
- Downloads:
- 0
- Uploads:
- 23
- Thanks:
- 4
- Thanked:
- 33 times in 1 posts
November 11, 2013 at 4:14 pm #511854Hi Luna,
Thanks for the follow-up. I was able to work out a solution for the first issue. I shared the tip here: http://www.joomlart.com/forums/topic/how-to-have-custom-field-included-in-job-alert/As for the second one, is there a timetable on that fix? I don’t know if it is a good idea to turn on the alerts system when it can still send out incorrect alerts. We are so close on this issue – it is 90% done, and I would love to see it finished!
brentwilliams2 Friendbrentwilliams2
- Join date:
- January 2013
- Posts:
- 231
- Downloads:
- 0
- Uploads:
- 23
- Thanks:
- 4
- Thanked:
- 33 times in 1 posts
November 11, 2013 at 9:30 pm #511891I think I may have tracked it down, but I would love for you all to confirm that this is what I should be doing. It looks like the function auto_check_jobalert_about_job is what starts the process of adding the job ids to each job alert. I found the following code in models/jajobs.php, which appears to run this function if the job is not a new job:
if ((($diff_cat != '') && ($diff_location != '')) || $old_title != $row->title || $old_job_detail != $row->job_detail || $old_job_type != $row->job_type) {
auto_check_jobalert_about_job($row, $diff_cat, $diff_location);
}Is this correct? Can I comment out this section to stop new additions to the job alerts on edited jobs?
Luna Garden ModeratorLuna Garden
- Join date:
- July 2011
- Posts:
- 2617
- Downloads:
- 80
- Uploads:
- 96
- Thanks:
- 78
- Thanked:
- 453 times in 425 posts
November 12, 2013 at 3:00 am #511910Hi BrentWilliams,
Thanks for sharing your tips. We appreciate your contribution to JA JobBoard.
<blockquote> I found the following code in models/jajobs.php, which appears to run this function if the job is not a new job:</blockquote>
Yes, it the code for Job Alert on edited jobs. You can comment out it to stop the alert. We’ll try to improve the Job Alert on next version.brentwilliams2 Friendbrentwilliams2
- Join date:
- January 2013
- Posts:
- 231
- Downloads:
- 0
- Uploads:
- 23
- Thanks:
- 4
- Thanked:
- 33 times in 1 posts
November 18, 2013 at 8:02 pm #512631For anyone else reading this, in my post above (#35), I edited out the part that creates a job alert on an edited job. I kept seeing these wrong alerts, however, and realized that you also need to make the same edit to the file in the administrator section
administrator/components/com_jajobboard/models/jajobs.php
1 user says Thank You to brentwilliams2 for this useful post
-
AuthorPosts
This topic contains 37 replies, has 3 voices, and was last updated by brentwilliams2 11 years ago.
We moved to new unified forum. Please post all new support queries in our New Forum