Skip to content
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

Title - Description after moving to YPTStorage #30

Open
GreenDragonTavern opened this issue May 16, 2022 · 33 comments
Open

Title - Description after moving to YPTStorage #30

GreenDragonTavern opened this issue May 16, 2022 · 33 comments

Comments

@GreenDragonTavern
Copy link

After moving video to YPTStorage, Title and Description change, why? Images included and only thing that changed was moving from local to remote storage.

LocalStorage
YPTStorage

@DanielnetoDotCom
Copy link
Member

the storage move does not suppose to make it happen. maybe you are outdated, and this issue is already fixed

but I saw a similar thing happen with the clone plugin before.

and I build it to help fix it.

https://github.com/WWBN/AVideo/blob/master/install/fixSlashN.php

this is a command-line script that you can also modify to fix the multiple slashes

DanielnetoDotCom pushed a commit to WWBN/AVideo that referenced this issue May 16, 2022
@DanielnetoDotCom
Copy link
Member

I just sent a modification to the fixSlashN.php file, I hope it works for you

@GreenDragonTavern
Copy link
Author

I am definitely current on all; AVideo, Encoder, and Storage. Original video stored locally was spot on, no issues, and all I did was move to YPTStorage. Is the multiple slash fix implemented on move to YPTstorage? Just did a test on a YouTube video that stored it directly on YPTStorage and the multiple slash issue is definitely there. Changed back to local storage and presto, no slash issue on same video.

@DanielnetoDotCom
Copy link
Member

can you reproduce the slash problem on demo site?

if works there means you are not on the latest. or the error is something else

@GreenDragonTavern
Copy link
Author

Just uploaded same video to your demo site but when I went to check it, could not log back in...

@GreenDragonTavern
Copy link
Author

Just tested video on your site. It was stored local, and I moved it to storage 3. Yes, demo site is having same slash issue after moving to remote storage.

@GreenDragonTavern
Copy link
Author

You should see two of the same (Moments w/ Michael Jaco~ Retired 35Year Navy Seal & CIA Op Plays Crystal Song Bowl on Casey Key, FL) on your demo site. One local, one remote storage. Local description is spot on, remote has slash issue...

@DanielnetoDotCom
Copy link
Member

Just making clear, this happens only when we move?

or does it happen when we import/upload a video?

@GreenDragonTavern
Copy link
Author

Both regarding YPTStorage. If I set YPTStorage as default and upload (meaning directly to YPTStorage) there's a slash issue. If I upload and store locally, there is no slash issue until after I move it to YPTStorage. That make sense?

@DanielnetoDotCom
Copy link
Member

I did a test on the demo site and I could not reproduce it.

both (Moments w/ Michael Jaco~ Retired 35Year Navy Seal & CIA Op Plays Crystal Song Bowl on Casey Key, FL) were already with the "\n" before I transfer

and after the transfer, I did not see any difference

also, I transferred the “2000 Mules” Trailer and it seems to be fine

chrome-capture-2022-4-16

@GreenDragonTavern
Copy link
Author

I just uploaded on your demo site the same video again (previous local version was gone), it is stored locally. Look at description, and then look at description on remote storage one you did...they are not the same. I can give you another that is full of ////////////////////////////// if you like but shouldn't matter. Description should be identical whether its local or YPTStorage and they're not the same.

@DanielnetoDotCom
Copy link
Member

where does this video come from?

is it a youtube video?

can you please share a link so I can make some tests?

@GreenDragonTavern
Copy link
Author

It's a YouTube video, here's link: https://youtu.be/XgB64_vJn1w

@GreenDragonTavern
Copy link
Author

Here's a really good example: https://youtu.be/ay_Eohsjn2o

Upload it from YouTube, local only. Look at title, description, it's exactly as is as seen on YouTube. Then transfer to one of your YPTStorage sites and see all the //////////////////////////////////////////

DanielnetoDotCom pushed a commit to WWBN/AVideo that referenced this issue May 16, 2022
@DanielnetoDotCom
Copy link
Member

Hi, I guess the last AVideo update fix the issue

@GreenDragonTavern
Copy link
Author

This issue is definitely not fixed using paid plugin YPTStorage.
AfterLatestPull

@DanielnetoDotCom
Copy link
Member

Did you test on demo?

Did you update your avideo with git pull?

@GreenDragonTavern
Copy link
Author

Yes, and Yes. Your demo site does exactly the same as my site. Upload a YouTube video and storage set to local, works fine and Title, Description are perfect. Move that video to YPTStorage and //// shows up. Change default storage to YPTStorage and upload same video, //// shows up. It does not matter if you move to YPTStorage or natively store to YPTStorage it is the same result, you get ////.

I have tested twice on your demo site, the behavior was the same both times, and is exactly the same behavior happening on my site.

@DanielnetoDotCom
Copy link
Member

I did a test and it was working

I am away from pc today I will check it again tomorrow

@GreenDragonTavern
Copy link
Author

Sounds like a plan, and thank you.

@DanielnetoDotCom
Copy link
Member

I just make more tests on the demo site.

I moved the video between storages

local to site, site to site, and site to local

also I cleared the cache to make sure it was working.

everything works as expected .

Sorry I am not able to reproduce this issue.

@GreenDragonTavern
Copy link
Author

Just did 2 quick YouTube uploads, one on storage, one on local...they "are not" the same on your demo site.

https://youtu.be/kYJ82izYLmc is URL

@GreenDragonTavern
Copy link
Author

From your demo site:

LocalStorage_Demo

YPTStorage_Demo

@GreenDragonTavern
Copy link
Author

1st image is local storage, 2nd is after moving to Storage 1 on your demo site.

@DanielnetoDotCom
Copy link
Member

I guess I am missing a step to test it.

I still cannot reproduce that.

@GreenDragonTavern
Copy link
Author

Download video 2 times (very small clip): https://youtu.be/kYJ82izYLmc

Keep one on local storage, and move one to any YPTStorage device on your demo system. Edit the one on local storage, it is exactly as seen on YouTube. Edit the one you moved to YPTStorage, it is a mess...

@DanielnetoDotCom
Copy link
Member

I see, the error happens when you edit a moved video?

@GreenDragonTavern
Copy link
Author

No, nothing to do with editing Title, Description. Just replicated error on your demo site, it's there now...one local, one YPTStorage.

YPTStorage_Demo2

@GreenDragonTavern
Copy link
Author

And this is what displays for video on local storage:

LocalStorage_Demo2

@DanielnetoDotCom
Copy link
Member

Do you mind recording a video reproducing the error?

I am still having a hard time to reproduce it

@GreenDragonTavern
Copy link
Author

I am heading to our data center shortly...will make a recording of process later today. My time is UTC-8, and you?

@DanielnetoDotCom
Copy link
Member

I am in GMT -3

I will probably be able to check it tomorrow only

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants