I had a bunch of small issues to get Freelink Prepopulate to work in Drupal 7, some variables that were not set and some other stuff. A couple of fixes came from http://drupal.org/node/1267934.

CommentFileSizeAuthor
#21 freelinking.tar_.zip24.03 KBmisc
#19 freelinking_prepopulate-1351932-19.patch7.87 KBmisc
#14 freelinking_prepopulate-1351932-14.patch7.99 KBdarrenmothersele
#9 freelinking_prepopulate-1351932-4.patch7.07 KBjzornig
#7 errorcode109.txt370 bytesAkra
#3 freelinking_prepopulate-1351932-3.patch10.5 KBmisc
#1 freelinking_prepopulate-1351932.patch8.86 KBmisc

Comments

misc’s picture

misc commented
StatusFileSize
new freelinking_prepopulate-1351932.patch8.86 KB

And here is the patch. I also needed to change in one of the plugins for freelinking.

misc’s picture

misc commented
Status: Needs work » Needs review

Forgott to change status (sorry, some day I will learn).

misc’s picture

misc commented
StatusFileSize
new freelinking_prepopulate-1351932-3.patch10.5 KB

Cleaned up the patch a little bit.

gbenciolini’s picture

gbenciolini commented

Excuse me my ignorance and my bad english.

How could i installa patch without a consolle? I have my blog on a hosting server.

Thanks!

misc’s picture

misc commented

There is no ignorance in a question :-)
If you look in the patch you could see where and what to change in you original file - so you could do that in an editor. A little bit tricky to get everything right, but it works. Also there are tools that could help you with that also.

gbenciolini’s picture

gbenciolini commented

Thanks a lot !!!!!

Akra’s picture

Akra commented
StatusFileSize
new errorcode109.txt370 bytes

Thanks for the patch fix most of my problem, but only have this error,
Notice: Undefined index: title en freelinking_prepopulate_node_settings() (línea 109 in freelinking_prepopulate\freelinking_prepopulate.module), well thx for ur time, and sry for my bad english

misc’s picture

misc commented
Status: Needs review » Needs work

I will look into that and do a new patch.

jzornig’s picture

jzornig commented
StatusFileSize
new freelinking_prepopulate-1351932-4.patch7.07 KB

I rerolled the #3 patch for the feb 21 dev

misc’s picture

misc commented
Status: Needs work » Needs review
Flandry’s picture

Flandry commented
Version: 7.x-3.1 » 7.x-3.x-dev

I applied the patch manually (nb i did not apply the multiline comment to hide the "unknown" chunk of code as the patch does) and it took care of the error messages i was getting when configuring Freelinking and when previewing a page that invoked freelinking. Something strange happened at first (illegal option) with the Advanced Settings checkbox under Createnode Plugin Settings, but after unchecking and then rechecking that option (Use the Linking Node for Create Node presets), it seems to be doing that.

I really like freelinking but the utility (for me, at least) was limited without the Prepopulate addition, and that is broken in the present release. Please apply the patch and tag a new release so others less inclined to apply patches can enjoy your great work!

misc’s picture

misc commented

Good point, we should prepare for a new release.

rishi_cs357’s picture

rishi_cs357 commented

Hey MiSc ...
I have applied #9 patch most of errors have been resolved now ...i am getting following errors....
Thanx in advance.Now what to do?

Notice: Undefined variable: object in freelinking_prepopulate_fields_from_page() (line 43 of ....../sites/all/modules/wiki/freelinking/modules/freelinking_prepopulate/freelinking_prepopulate.utilities.inc).
Notice: Trying to get property of non-object in freelinking_prepopulate_fields_from_page() (line 43 of ...../sites/all/modules/wiki/freelinking/modules/freelinking_prepopulate/freelinking_prepopulate.utilities.inc).

-----Rishi

darrenmothersele’s picture

darrenmothersele commented
StatusFileSize
new freelinking_prepopulate-1351932-14.patch7.99 KB

Rerolled the patch from #9 adding back in the object loading.

winklet’s picture

winklet commented

I'm not able to apply any of these patches! I am working from 7.x-3.x-dev. Is this the wrong version? I'm a newbie to applying patches and maybe need some help. I've been able to do it before but this time it simply will not do it.

misc’s picture

misc commented

How do you apply the patch?

winklet’s picture

winklet commented

I put the patch file in the freelinking module folder and then use Tortoise SVN to apply the patch.

misc’s picture

misc commented
Status: Needs review » Needs work

I see that the patch in #14 does not work, I will check it.

misc’s picture

misc commented
StatusFileSize
new freelinking_prepopulate-1351932-19.patch7.87 KB

Updated patch, it should now work. I apply patches with git (if I have downloaded the module via git): git apply -v patchname or with patch like patch -p1 < patchname. Both done in the module folder. (If you have curl and git you could do curl -0 patchurl | git apply -)

winklet’s picture

winklet commented

Thanks. I think I was able to apply this patch, however it has not solved any of the problems (multiple error notices) when trying to add freelinks in the page bodies. I think I am going to give up and find something other than Drupal for the wiki we want to set up.

misc’s picture

misc commented
StatusFileSize
new freelinking.tar_.zip24.03 KB

@winklet, I attached a patched version of the module. Test this if you like.

misc’s picture

misc commented
Status: Needs work » Needs review

Patch need review by the way :-)

misc’s picture

misc commented

Also @winklet, if you have problems, better give us the errors that you get, and we could together solve it. Just "has not solved any of the problems" helps nobody.

juampynr’s picture

juampynr commented
Title: Problems with Prepopulate » Several warnings with Freelinking Prepopulate submodule
Status: Needs review » Fixed

The issues reported here are very similar as the ones fixed at #1671418: Several E_NOTICE fixes at freelinking_prepopulate. I personally have reviewed little of Freelinking Prepopulate and it seemed that it had (and may still have) several bugs. In order to stabilise it I have created another issue to implement tests for it: #1673694: Add tests to Prepopulate submodule.

I have manually applied patch from comment #21. Many thanks to everyone!

http://drupalcode.org/project/freelinking.git/commitdiff/d1e9561

Status: Fixed » Closed (fixed)

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