-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Undefined index: vjs_sync #130
Comments
This part of the code did not change. |
Thanks. |
Then I need more details because I am unable to reproduce the issue.
|
No problem - just tell me what is needed |
|
Can you retry with the latest commit using latest master. |
Now i get
|
More investigation with an old server-backup where 2.8.b is working.
=> exiftool is installed and "green" in plugin-settings (like everything else)
|
I modify to exit on |
Now there is no error message! I tested it with a lot of videos that worked fine on 2.8.b |
This is expected. I remove all errors, However without any configuration you can't go any further. |
2.9.b release |
Conclusion: no error-messages any longer, but the plugin is not working as 2.8.b. Will go back to 2.8.b |
The page |
As i told you i did ONLY an update from 2.8.b to 2.9.a Is the configuration stored on disk or in the DB? |
Settings are stored in table config under vjs_conf. Changed values show up on website. That seems to work correct on 2.9.b But once again, after saving settings and so on...:
Admin-panel > Plugins > VideoJS > Synchronize > [select album and no simulation, same settings as above] > Submit
|
I just updated to 2.9.b and started to get these when I click Submit on the Synchronize tab. It was working fine on 2.9.a.
In config table, I saw Any ideas? |
I reworked the default sync_options for the photo, batch_manage and sync page. |
For part 1 via 243c0f9 solved For part2, we need save the custom binary part separately. I will instead move the binary configuration part to local config file
Custom binary path will need to set manually into the local config file |
Great. Looking forward to the new release. |
I recall why |
Custom binary path now need to set manually into the local config file local/config/config.inc.php.
Please test and feedback. |
I removed the old version and installed 2.9.b via GUI. Then replaced it with the latest master. First of all, what do you mean by "part1" and "part2"? Is my reported bug part1 and part2 chen5317's? Let's start with the first method to update fotos, select a set of videos and choose "Action:Videos":
Let's continue with the "Synchronize" method. I tried it without setting any $conf-variable in my config.inc.php, just to see what happens:
Another small bug i found: there is a setting on the bottom of the settings-page called "RESOLUTION" without a description. Changes here are not saved, it stays disabled. |
Thanks for the feedback. "RESOLUTION" description has been add in previous commit #125 , if you don't see it you are not using the latest version or you need to help translating new option in your language. |
I need to use custom binary path for mediainfo and I use avconv instead of ffmpeg. I don't know why it doesn't take my config. I set them to these
It tells me this.
I also have exiftool in my system so I tried setting |
Those config parameters refer to directory not to full binary path.
Now as you use avconv instead of ffmpeg, and ffmpeg is hard coded as in PWG, it is another problem.
|
Thanks for the next bugfix. As before i removed the plugin, installed 2.9.b and replaced it with master. Then disabled/enabled it. "RESOLUTION": not only the description is missing, changes are NOT saved. It always stays disabled. synchronization-method:
batch-manager-method:
|
You're right. If I specify the folder name instead of the full path, it works for mediainfo. The remaining issue is to be able to support avconv. For now, I can just set |
"RESOLUTION": it is pending a larger update so it is currently expected. synchronization-method:
batch-manager-method:
|
I updated the code to allow poster and thumbnail creation without parsing metadata via |
batch-manager-method:
synchronization-method:
|
synchronization-method: |
I will test it again... |
Tested again, and it still does not work. I removed plugin via GUI, installed 2.9.b via GUI, activated it. Then replaced the directory with the master. Synchronization results Edit: 18 videos are available in the selected album |
Same problem for me using fresh Piwigo install and GUI install of piwigo_videojs plugin v 2.9b
|
Seems i have the same problem, unable to synchronise (error message:admin_sync line 78 same as jessek09) I have updated the local configuration and added the binary files as advised but the problem persists. |
@bb1bsc: same problem that i described on my last post or same like jessek09? |
@Ewald342 Not Tested: Approach: My Steps: // picture_ext : file extensions for picture file, must be a subset of // file_ext : file extensions (case sensitive) authorized I have since sync 180 videos. The only problem i have is generating Posters for .MOV but thats a seperate issue. |
Will there be an official release with the changes and instructions? |
I was getting the following error with batch_manager:
it was solved by the following DB updated: |
You made my day, thank you |
I get the following Notice when I click on Submit on the Synchronise tab: Undefined index: vjs_sync in /home/stevestuff/public_html/piwigo/plugins/piwigo-videojs/admin/admin_sync.php on line 78 Thereafter the the form no longer contains any information. I found that that param vjs_sync was set to the value"N;" in the config database table. This is supposed to be a serialized array and being that it is not I get: Warning: array_merge(): Argument #1 is not an array in /home/stevestuff/public_html/piwigo/plugins/piwigo-videojs/admin/admin_sync.php on line 53 when I click on the Synchronise tab. Deleting the vjs_sync record puts me back where I started. I found that I could work around this problem by commenting out line 78 of the /piwigo/plugins/piwigo-videojs/admin/admin_sync.php file, then clicking Submit, and then re-enabling line 78. I think the vjs_sync is supposed to be initialized when the plugin is activated and it's not. I noticed that vjs_conf is initialized when the plugin is activated.. |
Fixed the problem by setting paths as mentioned by @bb1bsc and replacing plugin files installed from the GUI with GitHub master source code. |
Server: Debian Jessie 8.11
PHP: PHP 5.6.36-0+deb8u1
Apache: 2.4.10 (Debian)
videojs: 2.9a
When i try to update a video manually (Admin panel > Photos > Batch > select a video > chose "Video" and use the following settings:
Synchronize metadata: checked
Create a poster: checked
Overwrite: checked
Add film effect: checked
Create thumbnail: unchecked
I get this errors:
Worked for years, but now something is broken. Don't know if it was the last apt-get update or the update to plugin 2.9.a
The text was updated successfully, but these errors were encountered: