Joomla-Bible-Study / Proclaim

Joomla Proclaim Component - Christian Web Ministries
https://www.christianwebministries.org/
GNU General Public License v3.0
10 stars 9 forks source link

Images do not save for podcast #942

Closed tomfuller2 closed 1 year ago

tomfuller2 commented 1 year ago

Selected an image from the rotating images folder. When saving got this error:

Save failed with the following error: Data too long for column 'image' at row 1

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Additional context Add any other context about the problem here.

bcordis commented 1 year ago

@tomfuller2 Are we supporting wanting to use external images here or just local ones? Right now, I can save select local images in the media bin.

This may need to be sorted out, as we want to use an image folder, not the media folder.

bcordis commented 1 year ago

Found a problem with Joomla 4 using the FileSystem - Local as it needs to add the file location of images. This allows the user to select the right images on the podcast page.

tomfuller2 commented 1 year ago

We’re you able to fix?

On Fri, Apr 28, 2023 at 6:45 AM Brent Cordis @.***> wrote:

Found a problem with Joomla 4 using the FileSystem - Local as it needs to add the file location of images. This allows the user to select the right images on the podcast page.

— Reply to this email directly, view it on GitHub https://github.com/Joomla-Bible-Study/Proclaim/issues/942#issuecomment-1527589609, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJYYXKGR4R3442I7NW3VB3XDPCYHANCNFSM6AAAAAAXOFLIOY . You are receiving this because you were mentioned.Message ID: @.***>

-- Tom Fuller, APR

Founder and Principal

Imagine Public Relations

www.ImaginePublicRelations.com 971-832-9494 | 503-550-4067 @.*** www.ImagineVisualProductions.com https://www.imaginevisualproductions.com/ [image: facebook] https://www.facebook.com/imaginepublicrelations [image: twitter] https://twitter.com/thomasefuller [image: linkedin] https://www.linkedin.com/in/tomfuller/

bcordis commented 1 year ago

Working on the final fix now

bcordis commented 1 year ago

Right now, I can't see any problem saving the podcast images. Please try again and verify what is up to date and what version of PHP you are testing with.

bcordis commented 1 year ago

@tomfuller2 Need your replay

tomfuller2 commented 1 year ago

Need to do more research. Right now I get an error: Save failed with the following error: Data too long for column 'image' at row 1 Tom Fuller, APR www.ImaginePublicRelations.com www.ImagineVisualProductions.com 971-832-9494

On Mon, Jun 19, 2023 at 12:17 PM Brent Cordis @.***> wrote:

@tomfuller2 https://github.com/tomfuller2 Need your replay

— Reply to this email directly, view it on GitHub https://github.com/Joomla-Bible-Study/Proclaim/issues/942#issuecomment-1597645093, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJYYXNHPX75KQZXIIAVL7DXMCQTRANCNFSM6AAAAAAXOFLIOY . You are receiving this because you were mentioned.Message ID: @.***>

bcordis commented 1 year ago

@tomfuller2 Looking through the Db looks like a lot of core DB has changed to 255 characters. This may facilitate the fix to the problem

Try changing the DB character limit. and lets see if this fixes the problem.

bcordis commented 1 year ago

Fixed in Dev