| [21:07:14] | * JustTB has joined #farmos |
| [22:01:31] | * JustTB has quit (Quit: Leaving.) |
| [04:34:02] | * konstamtin has joined #farmos |
| [04:34:52] | <konstamtin> | Hi everybody. Trying to install and set-up FarmOS - have several issues and questions |
| [04:37:29] | <konstamtin> | PHP Version 5.4.20, FreeBSD. Onw hosting platform, Database - MySQL. During installation while configuring modules - had several subsequent errors about impropes use of empty function |
| [04:38:07] | <konstamtin> | After minor changes in related modules (commentsing some fool-profing blocks of code) I have it installed. |
| [04:39:24] | <konstamtin> | Now farmOS is works, but I have issue with GoogleMaps integration. I've got API key, put it into setting but still have error with map |
| [04:40:07] | <konstamtin> | In console - NoAPIkey error |
| [04:41:08] | <konstamtin> | Any ideas where I can check the settings. How I can ensure that API key entered in setting successfully transferred into code in modules? |
| [04:46:26] | <konstamtin> | I have created temporary FarmViewer account for our discussion. URL: farm.kostello.ru Login: test PWD: test!123 |
| [04:54:59] | <konstamtin> | Also after installation - I have updated the opanlayers and feeds modules - they were marked with available updates in Modules configuration |
| [05:22:43] | <konstamtin> | And yes, I used pre-packaged Drupal distribution from https://www.drupal.org/project/farm |
| [08:17:13] | * trentf has joined #farmos |
| [11:49:26] | * JustTB has joined #farmos |
| [12:02:06] | <mstenta_> | Hi konstamtin |
| [12:03:00] | <mstenta_> | Regarding improper use of empty() function - I've never experienced this. Can you tell me which pieces of code you had to comment out? |
| [12:03:42] | <mstenta_> | Your PHP version seems fine... I'm not sure why you would be getting that error. I haven't tried installing on FreeBSD, but I don't think that would be different. Are you using Apache? |
| [12:04:24] | <mstenta_> | Regarding broken Google maps: you should undo your Openlayers module upgrade. |
| [12:04:46] | <mstenta_> | I'm not sure if that is the original cause... but it will cause other issues regardless. |
| [12:05:06] | <mstenta_> | The newer version of farmOS I am working on will update Openlayers, but there are some required patches and other updates necessary. |
| [12:05:24] | <mstenta_> | Simply updating the module without applying the patches, and without making the necessary updates to farmOS itself will break your maps. |
| [12:05:28] | <mstenta_> | That might be what happened. |
| [12:06:20] | <mstenta_> | The best way to roll back is to download the 7.x-1.0-beta15 zip file from http://drupal.org/project/farm, unzip it in another folder, and get the /profiles/farm/modules/contrib/openlayers folder - use that to replace your current folder. That will include the patches. |
| [12:07:32] | <mstenta_> | The Feeds module upgrade should be harmless - but in general you shouldn't need to worry about updating individual modules. When the next version of farmOS is available, it will include all the updates. |
| [12:08:07] | <mstenta_> | I try to make sure that all the modules work together without issues. Openlayers is a very special case in farmOS because it requires a number of patches in order to work correctly. |
| [12:08:28] | <mstenta_> | After you roll back your Openlayers module code, clear your cache. |
| [12:08:44] | <mstenta_> | Then, go to /admin/config/farm/map - you should see a field there for the Google Maps API key. |
| [12:09:06] | <mstenta_> | If you don't see your key there, enter it and save (and let me know - hopefully there is not a bug during installation in saving that value). |
| [12:09:34] | <mstenta_> | If you do see a value, and it's still not working, you can confirm that it is being used in the Openlayers configuration by doing the following: |
| [12:09:42] | <mstenta_> | 1) go to /admin/modules, and enable the Openlayers UI module |
| [12:10:23] | * efraim has quit (Remote host closed the connection) |
| [12:10:37] | * efraim has joined #farmos |
| [12:10:37] | <mstenta_> | 2) go to /admin/structure/openlayers/sources/list/farm_map_source_google_hybrid/edit/options - and you should see the key there as well. |
| [12:11:04] | <mstenta_> | The form at /admin/config/farm/map is a shortcut to fill in that field in the Openlayers Source configuration |
| [12:11:39] | <mstenta_> | If the key is in both places, but still isn't working - make sure you have set it up properly in Google (and copied it properly, with no spaces to the left/right of it) |
| [12:12:04] | <mstenta_> | Instructions are here: http://farmos.org/hosting/googlemaps/ |
| [12:12:13] | <mstenta_> | Good luck! Let me know if you don't figure it out. |
| [12:12:32] | <mstenta_> | I will be traveling tomorrow, and won't be back for the rest of the week. I won't be on IRC. |
| [12:12:43] | <mstenta_> | If you want to file a bug report on Github, I will see that and respond when I can. |
| [12:17:35] | <konstamtin> | Hi Michael, thanks for your instruction. I will try to do everything you've recommended and report |
| [12:24:51] | <konstamtin> | As for empty() function I can try to create another instance on my hosting platform in order to reproduce the erroe |
| [12:32:55] | <konstamtin> | Ok, openlayers is copied from original source, cache cleared. As for /admin/config/farm/map - that's could be problem. I can see only checkbox for "Show dashboar map". I've entered Google Key in /admin/config/content/geocoder section |
| [12:38:36] | <konstamtin> | farm_map_source_google_hybrid/edit have no Google Map API Key field at all |
| [12:47:20] | <mstenta_> | konstamtin: Hmm - are you using farmOS 7.x-1.0-beta15? |
| [12:47:27] | <mstenta_> | You may be on an older version |
| [12:48:41] | <mstenta_> | konstamtin: oh wait... nevermidn - I'm wrong |
| [12:49:05] | <mstenta_> | Sorry - I gave you instructions for the development version - not the latest release |
| [12:49:36] | <mstenta_> | I added the shortcut API Key field to /admin/config/farm/map recently - but it hasn't been released yet - sorry that was my mistake |
| [12:50:05] | <mstenta_> | With that in mind, you should just enter the API key in /admin/structure/openlayers/sources/list/farm_map_source_google_hybrid/edit/options |
| [12:50:36] | <mstenta_> | The form at /admin/config/content/geocoder is actually a different thing - it is for the Geocoder module - not for the Openlayers module |
| [12:51:01] | <mstenta_> | (That was a common mistake people made, which is why I added the shortcut form - that will be available in the next release) |
| [12:51:05] | <mstenta_> | Hope that helps! |
| [12:52:26] | <mstenta_> | I am curious about the empty() function issues... if you do set up another instance let me know what you find. |
| [12:52:41] | <mstenta_> | I'm curious if it was a PHP error/warning/notice you saw - or if it came from something else |
| [13:25:13] | * JustTB has quit (Quit: Leaving.) |
| [13:43:47] | <konstamtin> | farm-7.x-1.0-beta15-core |
| [13:44:59] | <konstamtin> | as for empty - it was php errors |
| [13:58:50] | <mstenta_> | konstamtin: thanks |
| [13:59:04] | <mstenta_> | i've never seen a php error about "improper use of empty()" |
| [14:00:09] | <konstamtin> | it's not the exact error description. It's was something about that empty can't use not defined parameter |
| [14:00:22] | <konstamtin> | When I reproduce - I'll let you know |
| [14:06:35] | <konstamtin> | Michael, thanks a lot - Maps works now! |
| [14:06:51] | <mstenta_> | konstamtin: great! |
| [14:06:56] | <mstenta_> | glad to hear it's working! |
| [14:07:06] | <mstenta_> | yes, please do forward along the specific error message when you can! thanks! |
| [14:07:42] | <konstamtin> | Honestly - it was not obvious for me to find and setup. Maybe next release it will be easier to navigate as you said |
| [14:07:55] | <mstenta_> | The Google Maps API? |
| [14:08:01] | <konstamtin> | yes |
| [14:08:06] | <mstenta_> | Yes - I've improved that in the next release - it will be a step in the installation |
| [14:08:34] | <mstenta_> | So you just copy and paste your key when you are setting up your other initial configuration options, and it gets filled in in the background |
| [14:14:42] | <konstamtin> | I also have some issue with Mushrooms module. I've created Substrate type, Species/Breeds records. When I try to create substrate record - it doesn't display relevant fields for Name, Substrate type and Species/Breeds. Only Location filed is displayed. So when I specify Location and try to save - the error displayed that I have to enter Name |
| [14:15:47] | <mstenta_> | konstamtin: hmm - that might be because the mushroom module has been updated for the new version of farmOS (which hasn't been released yet) |
| [14:16:14] | <mstenta_> | konstamtin: are you comfortable with Git? I can give you a specific commit ID to checkout for the mushroom module that should work with the beta15 of farmOS |
| [14:16:53] | <konstamtin> | I think I can try |
| [14:17:23] | <mstenta_> | ok - it's pretty easy if you have Git |
| [14:25:42] | <mstenta_> | Here is the commit you need: c53b2e5a320df0aa7a9738178a9286dc10be857d |
| [14:25:55] | <mstenta_> | It is actually the one right before the latest commit |
| [14:48:21] | <konstamtin> | ok, when I try to install new instance of farmOS the following error displayed during install profile: "An AJAX HTTP error occurred. HTTP Result Code: 500 Debugging information follows. Path: http://farm1.kostello.ru/install.php?profile=farm&locale=en&id=1&op=do StatusText: Internal Server Error ResponseText:" |
| [14:48:32] | <konstamtin> | server logs: PHP Fatal error: Can't use function return value in write context in /www/kostello/users/kostello-farm1/www/htdocs/profiles/farm/modules/farm/farm_import/farm_import.feeds_importer_default.inc on line 217, referer: http://farm1.kostello.ru/install.php?profile=farm&locale=en&op=start&id=1 |
| [14:48:49] | <mstenta_> | right now farm_mushroom (and a few others) don't have "official" releases... so they aren't pegged to a specific version of farmOS, even though they need to be... i will try to make that a priority moving forward, adn release versions of them that depend on a specific version of farmOS, to avoid that issue |
| [14:49:15] | <mstenta_> | konstamtin: ok thanks... let me take a look at that file right now... one moment... |
| [14:50:05] | <konstamtin> | Server configuration farm.kostello.ru/info.php |
| [14:50:39] | <konstamtin> | I had about 10 similar errors during installation in different modules |
| [14:51:01] | <mstenta_> | ok - yea i see why that might be a problem |
| [14:52:43] | <mstenta_> | i think i have a fix for it |
| [14:52:46] | <mstenta_> | can you test it for me? |
| [14:52:51] | <konstamtin> | yes |
| [14:53:18] | <mstenta_> | https://pastebin.com/2ztgGdcK |
| [14:53:34] | <mstenta_> | replace the whole farm_import_add_importer_fields() function with that |
| [14:54:23] | <mstenta_> | in [webroot]/profiles/farm/modules/farm/farm_import/farm_import.feeds_importer_default.inc |
| [15:02:41] | <konstamtin> | ok, now the following in error log Can't use function return value in write context in /www/kostello/users/kostello-farm1/www/htdocs/profiles/farm/modules/farm/farm_import/farm_import.feeds_importer_default.inc on line 228 |
| [15:03:15] | <konstamtin> | actually the same - but row number is shifted due to new function code |
| [15:04:54] | <mstenta_> | ok.. hmm looking... |
| [15:06:24] | <mstenta_> | ah ok... one sec... |
| [15:06:31] | <mstenta_> | (thanks for helping to test this) |
| [15:09:17] | <mstenta_> | konstamtin: hmm - actually - what does that line look like in your code? |
| [15:10:20] | <mstenta_> | I did find that there were some similar lines in farm_import.feeds_tamper_default.inc (not the same file) - those should probably be fixed too and may be part of the issue |
| [15:11:23] | <konstamtin> | if (!empty(field_info_instance($type, $field, $bundle))) { |
| [15:12:50] | <mstenta_> | oh - that's not what was in my paste... was it? |
| [15:12:57] | <mstenta_> | are you sure you pasted correctly? |
| [15:13:03] | <mstenta_> | that line should have changed |
| [15:13:49] | <mstenta_> | i made two changes in my fix |
| [15:13:58] | <mstenta_> | because there were two lines that were trying to do the same thing |
| [15:14:45] | <mstenta_> | replace the whole function with the contents of my paste |
| [15:15:02] | <konstamtin> | ok, I have git installed and configured wiith basics. Which github url I have enter to get the project clone? |
| [15:15:22] | <mstenta_> | and when you're done with that, before you run another test... also replace the entire contents of farm_import.feeds_tamper_default.inc with this: https://pastebin.com/N3zcbD8i |
| [15:17:29] | <mstenta_> | clone farm_mushroom from: https://github.com/farmOS/farm_mushroom.git |
| [15:17:41] | <mstenta_> | and after you have it cloned, go into the directory in your terminal and run: |
| [15:17:52] | <mstenta_> | AH BUT! |
| [15:17:56] | <mstenta_> | before you do that... |
| [15:18:05] | <mstenta_> | make sure that you uninstall the module from farmOS first |
| [15:18:15] | <mstenta_> | that means first disable it in /admin/modules |
| [15:18:33] | <mstenta_> | and then uninstall it in /admin/modules/uninstall |
| [15:19:03] | <mstenta_> | after you do that... clone the repo... and then go into the module directory and run the following command: |
| [15:19:15] | <mstenta_> | git reset --hard c53b2e5a320df0aa7a9738178a9286dc10be857d |
| [15:22:35] | <konstamtin> | ok, now we are progressing - next module /farm/modules/farm/farm_log/farm_log_movement/farm_log_movement.module on line 118 |
| [15:22:54] | <mstenta_> | ok |
| [15:23:03] | <mstenta_> | so i figured out why i haven't experienced this issue before too... |
| [15:23:17] | <mstenta_> | it is because PHP 5.5 allows that kind of code, and I've been testing on PHP 5.6 |
| [15:23:31] | <mstenta_> | would it be possible for you to upgrade to PHP 5.6? |
| [15:24:15] | <mstenta_> | there are probably going to be a lot of instances of that issue... |
| [15:25:11] | <mstenta_> | i think it will make more sense for me to fix the requirements on http://farmos.org/hosting/installing/ to say PHP 5.5+ |
| [15:27:19] | <konstamtin> | 5.5 is only available for qiuck resolution - just changing the version in htaccess file. Newer versions - only with special request to hosting support |
| [15:28:23] | <konstamtin> | after changes in htaccess info.php sows that PHP version 5.5.5 |
| [15:28:32] | <mstenta_> | ok great - that should solve your problems |
| [15:28:43] | <mstenta_> | i don't think you need 5.6 - just 5.5 |
| [15:29:32] | <mstenta_> | i am going to update farmOS.org requirements... it will be hard to find all the places in code that need to be fixed to support 5.4 |
| [15:30:38] | <konstamtin> | yes, after version changed - installation finished fine |
| [15:32:19] | <mstenta_> | great! |
| [15:32:28] | <mstenta_> | thanks for helping to figure that out - i wasn't aware of that limitation |
| [15:34:04] | <mstenta_> | i updated farmOS.org |
| [15:34:21] | <konstamtin> | thanks for your great project FarmOS - I hope it helps me a lot |
| [15:34:41] | <mstenta_> | me too! |
| [15:34:48] | <mstenta_> | lots of improvements coming in the next version too... |
| [15:35:05] | <mstenta_> | i announce new released via my farmier.com newsletter if you would like to get an email when it's ready |
| [15:35:56] | <konstamtin> | yes, it will be great |
| [15:36:52] | <konstamtin> | I'm going to share my farmOS experience on upcoming meeting of farmers in Moscow, Russia |
| [15:37:26] | <konstamtin> | Not many of them speaks English and able to cinfigure whatever site |
| [15:37:41] | <mstenta_> | great! |
| [15:37:50] | <mstenta_> | ah yea - i am looking forward to adding translations |
| [15:37:59] | <mstenta_> | it is technically possible in Drupal 7 - but it will be easier in Drupal 8 |
| [15:38:08] | <konstamtin> | yes, I can contribute with translation |
| [15:38:14] | <mstenta_> | that would be great! |
| [15:38:34] | <mstenta_> | i plan to announce a translation initiative via the newsletter as well - to ask for help |
| [15:49:44] | <konstamtin> | As for mushrooms - the problem the same after update |
| [15:50:48] | <konstamtin> | I've got new version, updated the files, re-installed the module, cleared cache' |
| [15:51:41] | <konstamtin> | Only location and Status field are visible on "add mushroom substrate" form |
| [15:52:04] | <mstenta_> | did you check out the specific commit? |
| [15:52:35] | <mstenta_> | you actually need a version that came right before the latest version, not the latest |
| [15:53:35] | <mstenta_> | inside the farm_mushroom directory, you need to run the following command: |
| [15:53:36] | <mstenta_> | git reset --hard c53b2e5a320df0aa7a9738178a9286dc10be857d |
| [15:54:10] | <mstenta_> | you can try that and clear the cache (without uninstalling and reinstalling) and see if that works... but you may need to uninstall/reinstall |
| [15:56:27] | <konstamtin> | yes i did git reset. and recreated the module after it. Should I use git commit also? |
| [15:58:55] | <mstenta_> | nope |
| [15:58:56] | <mstenta_> | hmm |
| [15:59:12] | <mstenta_> | can you take a screenshot of the add mushroom form? |
| [15:59:16] | <mstenta_> | and post it to imgur.com |
| [16:10:35] | <konstamtin> | https://imgur.com/4IpjIBh |
| [16:20:20] | <mstenta_> | konstamtin: hmm strange |
| [16:20:27] | <mstenta_> | does the same thing happen on any other asset types? |
| [16:20:31] | <mstenta_> | or just mushrooms? |
| [16:20:51] | <konstamtin> | I discovered it only for mushrooms |
| [16:21:22] | <konstamtin> | I haven't try to create any type of records in FarmOS yet |
| [16:23:31] | <mstenta_> | you could try going to /admin/structure/features, and clicking on the Contrib tab |
| [16:23:39] | <mstenta_> | and see if the mushroom feature says "Overridden" |
| [16:30:17] | <konstamtin> | No State is Default |
| [16:32:41] | <konstamtin> | and Signature is Unavailable |
| [16:32:49] | <mstenta_> | huh. very strange |
| [16:33:11] | <mstenta_> | i will have to test that myself on a new install and see if i can replicate it |
| [16:33:24] | <mstenta_> | i have to go now, though |
| [16:33:51] | <mstenta_> | if you have a moment, could you create a bug report on https://github.com/farmOS/farmOS/issues so I don't forget? |
| [16:34:41] | <konstamtin> | ok, I also have to go. It was a pleasure to talk with you today. By the way - nice pictures on your site :) |
| [16:34:59] | <mstenta_> | thank you! :-) |
| [16:35:03] | <mstenta_> | and thanks again for the help! |
| [16:35:07] | <mstenta_> | talk to you again soon i hope! |
| [16:41:44] | * konstamtin has quit (Ping timeout: 260 seconds) |
| [17:04:39] | * trentf has quit (Remote host closed the connection) |
| [17:32:14] | * trentf has joined #farmos |