Patch for Job Search 6.x-1.x-dev - User profile updates and Views 2 integration

Thank you so much for this.

Can people please test this patch and verify that it works?

Hey guys!

I've just tested the patch, all seems to work properly, there are maybe use cases I didn't checked.

I corrected the patch because path were absolute, so I make them relative.

Thnak you jrosen
Jérémy

Just forgot to update status!

Jérémy

just updating the title to reflect a more descriptive meaning

When I originally posted the patch, I forgot to mention that I included another feature I didn't mention:

• The "Please Create a Resume to Apply Link" does not bring the user to the "Create Content" page.

  It loops through the assigned resume content-types (the ones assigned in admin/settings/job), and the first content-type in the list is automatically redirected to by going to "node/add/".

  So for instance, if you have a content-type called "cv" checked off in the list in admin/settings/job, then "Please Create a Resume to Apply Link" will redirect to "node/add/cv" instead of "node/add".

This feature is one that was requested in Issue #381462: Apply for Job Needs to Send User to Create Resume Form.

I didn't actually mean to include it in my patch, I meant to include it in a separate patch at a later time. So anyway, not sure if I sure mark issue #381462 as a duplicate or not.

Jason

This patch is not working for me. PS. I used Eclipse for it.

Please be more specific... what is not working? What errors or symptoms are you encountering?

After patching I`ve just noticed that there is no views integration.

After patching, do you have the file: "jobsearch/views/job.views.inc"? This should have been created by the patch.

Also the following lines should be in your job.module:

function job_views_api() {
return array(
'api' => 2,
'path' => drupal_get_path('module', 'job') .'/views',
);

If these are there, then you should have Views 2 integration.

Hi jrosen,
Thank you for your response. Everything is there as stated, but I still can't find new views. I don't know what I`m doing wrong.
Please find attached images.

Hi. Sorry, I'm new to patching so I'm not sure how to debug this. The patch file is being run from the jobsearch module folder.

I get the following error when trying to patch the module...

patch < jobsearch.patch
patching file README.txt
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file README.txt.rej
can't find file to patch at input line 16
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|diff -Nurp access/job_access.info jobsearch/access/job_access.info
|--- access/job_access.info 2009-01-17 04:08:22.000000000 -0800
|+++ jobsearch/access/job_access.info 2009-01-23 18:24:12.000000000 -0800
--------------------------
File to patch:

The README.txt.rej file contains the following...

***************
*** 1,6 ****
- $Id: README.txt,v 1.2 2009/01/02 02:49:22 kbahey Exp $

- Copyright 2007-2009 http://2bits.com

Description
-----------
--- 1,6 ----
+ $Id: README.txt,v 1.1 2007/06/24 00:23:05 kbahey Exp $

+ Copyright 2007 http://2bits.com

Description
-----------

OK I think i got it. I placed the patch file (the original file not the one updated with relative links) in the module directory and ran it with the following...

patch -p8 < jobsearch.patch

And got the following response...

missing header for unified diff at line 4 of patch
patching file access/README.txt
missing header for unified diff at line 16 of patch
patching file access/job_access.info
missing header for unified diff at line 38 of patch
patching file access/job_access.module
missing header for unified diff at line 74 of patch
patching file access/resume_access.info
missing header for unified diff at line 96 of patch
patching file access/resume_access.module
missing header for unified diff at line 133 of patch
patching file job.info
missing header for unified diff at line 154 of patch
patching file job.install
missing header for unified diff at line 218 of patch
patching file job.module
missing header for unified diff at line 891 of patch
patching file resume.info
missing header for unified diff at line 910 of patch
patching file resume.module
missing header for unified diff at line 960 of patch
patching file translations/ru.po
missing header for unified diff at line 970 of patch
patching file views/job.views.inc
missing header for unified diff at line 1149 of patch
patching file views/job.views_default.inc
missing header for unified diff at line 1240 of patch
patching file views/job_views_handler_field_applicant.inc

The files seem to have patched fine (according to the checks in reply #9 above) but the views aren't appearing in my views list.

Am I missing something?

OK well I'm a little embarrassed but thought I'd post in case it helps someone else. While I was testing the patch I kept a copy of my old module in the module folder as _jobsearch but drupal found this first and used it instead of my patched module. Removed this and the views appear.

So everything looks good apart from the 'missing header for unified diff at line...' when patching.

@adpo: please check the files after you have patched them as per my comment #9 in this thread. Also, after running the patch make sure you run update.php found in /admin/build/modules. You may also have to clear the Drupal cache by going to admin/settings/performance to clear the Views 2 cached views info.

please find attached jobsearch module after patching, I'am sure that your files has been updated OK. But maybe there is something else... what I did wrong.

@adpo: you have the views files in "jobsearch/jobsearch/views". The views folder should be here: "jobsearch/views"

When I applied the patch, I was in a directory outside the "jobsearch" directory. Could be you were inside the "jobsearch" directory and your patch program created a new "jobsearch/jobsearch" subdirectory.

Move the "views" folder up one level, use the Drupal update.php URL, clear your Drupal cache, and you should be up and running to use the Views 2 integration. (check #14 above for details on the update.php URL and clearing your Drupal cache).

Thank you very much. Your patch is working now.

hi where do i attach this patch to do i just insert it inside the jobsearch module folder and upload

folowed by update.php

never applied one before

Please see the following for guidance on applying patches:

For Windows: http://drupal.org/node/60179

For Mac OS X: http://drupal.org/node/60818

Maybe it's just me, but I can't get jchatard 's updated patch to work on Mac OS X (I followed the instructions from the handbook page).

We ran the patch, it added the views folder, but there appear to be no new views.

Can you issue a patched dev version for testing? Or export the views and make them available for us to import?

What patched dev version are you asking for? I have run the patch and have no problems.

Please be more specific as to what you need. I can export the views, but they won't work if the rest of the Views 2 integration code is not available because the rest of the code defines the fields, joins, and relationships that the exported views use.

Can you take a look inside the new views folder and tell me what files are inside? They should be:

• job.views.inc
• job.views_default.inc
• job_views_handler_field_applicant.inc

If these files are not in there, then let me know where they ended up so I can fix the patch. But to get it working, you can move those files into the views folder that was created, if they are somewhere else.

Let me know how it goes.

Was this included in the newest release? I don't see the views folder in the newest release.

When I apply the patch there are many errors. But it looks like it is only the .info files

$ patch -p0 < jobs*h
patching file access/README.txt
patching file access/job_access.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file access/job_access.info.rej
patching file access/job_access.module
patching file access/resume_access.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file access/resume_access.info.rej
patching file access/resume_access.module
patching file job.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file job.info.rej
patching file job.install
patching file job.module
patching file resume.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file resume.info.rej
patching file resume.module
patching file translations/ru.po
patching file views/job.views.inc
patching file views/job.views_default.inc
patching file views/job_views_handler_field_applicant.inc

Chris

also, the views do not show up.

Can we get this patch rolled into the Dev version for Drupal 6.x? It seems that others are having a problem applying the patch.

#22 jrosen -- I just saw this response; sorry for the delay. I'll check and reply/update.

#26 GreyHawk -- Yes that would be amazing. I've been patiently waiting and hoping for the Views 2 integration into the main branch.

Hi, I'm using Job Search now and wanted to mention something about the jobsearch.zip file above. I've been having problems seeing Job applications and decided to compare these files to the developed version (http://drupal.org/project/jobsearch).

I noticed that in the Translations folder, there is a file called ru.po. While in the developed version in the Translations folder, there is ru.po and es.po. Is this latter file suppose to be included in the jobsearch.zip package? Could this be why others are having problems too?

- john

@johnlee80 (#28):

The jobsearch.zip in comment #15 is not a usable patched version... he attached it so I could take a look and see why his was not working.

I will attach a working version in the near future. For now, please try to apply the patch yourself.

Any more info on the new patch?

Any chance somebody could wrap up the patch into new module release? I'm lost on this thread as to what to use. Thanks a TON.

#27 flamingpear -- I'm waiting for my cohort to return from vacation so I can be sure to test the right env (we have a couple testing setups); I hope to get back to it this week (before I forget what I was doing). ;)

I've been trying the patch and getting errors (mac terminal.)

Hoping somebody could post updated module .zip.

hi and thanks...

please find attached the latest dev version, as of this post (6.x-1.x-dev 2009-Apr-22)-
with your patch applied, ready to be installed, simple.

tested all functions and views integration...
seems to be working fine, with no issues or errors yet.

thanks for your patch / work... i applied this for a client-
perhaps it may be of use to others as well.

vincent, in buffalo
http://basicmagic.net

@basicmagic.net:

Thanks for posting the merged code w/patch. I'll take a look and see that it matches my version to make sure I haven't made any changes since.

Jason

subscribing...

any progress? subscribe

Tested basicmagic.net's module (#34), and it seems to be working well for our needs - no noticeable issues. Thanks!

Really hope to see these features applied to the 6.x DEV version. I ended up going with the code above for now since custom views are a requirement, and views support seems to be broken in the current 'official' DEV version of jobsearch.

.

I committed the patch to the -dev tarball, which should be created in a couple of hours.

Please test and report any issues you find as new issues.

Automatically closed -- issue fixed for 2 weeks with no activity.