?
avatar dkanchev
dkanchev
15 Aug 2016

Steps to reproduce the issue

Create a new com_newsfeeds feed and add a very long feed link to the link menu option during the creation. You may test with this one:

http://api.prosperent.com/api/search?api_key=55d3c3bf238c019a84cc3a6c6e57ab7c&query=&enableQuerySuggestion=1&page=1&location=http%3A%2F%2Fprosperent.com%2Faccount%2Fgetlinks%2Findex%2FfilterCategory%2AnotherCategory%2Test

Expected result

The whole link should be saved.

Actual result

The link is shortened to maximum 200 chars and then saved.

I tested this both on the Joomla.com platform and on a separate hosting server on which I installed the latest Joomla! version - the problem was present on both.

avatar dkanchev dkanchev - open - 15 Aug 2016
avatar jeckodevelopment
jeckodevelopment - comment - 15 Aug 2016

@dkanchev i'm not sure that the above linked is a RSS feed.
I see a JSON output, not a feed.

avatar dkanchev
dkanchev - comment - 15 Aug 2016

I don't have the whole valid link. The component just cuts it down to the following 200 chars:

http://api.prosperent.com/api/search?api_key=55d3c3bf238c019a84cc3a6c6e57ab7c&query=&enableQuerySuggestion=1&page=1&location=http%3A%2F%2Fprosperent.com%2Faccount%2Fgetlinks%2Findex%2FfilterCategory%2

I simply added more characters when I opened the issue, so that you can copy and paste it and see that it is cut. The valid link is longer and I don't have it.

avatar jeckodevelopment
jeckodevelopment - comment - 15 Aug 2016

But as far as i can see it's not a RSS feed, so it's almost normal that it doesn't work.

avatar brianteeman
brianteeman - comment - 15 Aug 2016

Probably a limit on the field size in the database?

avatar dkanchev
dkanchev - comment - 15 Aug 2016

You may check this video - to me it looks like a Joomla! limitation/problem:

http://screencast-o-matic.com/watch/cDj6rTi7wq

avatar jeckodevelopment
jeckodevelopment - comment - 15 Aug 2016

yes, i guess that it's a limit on the field.

avatar ot2sen
ot2sen - comment - 15 Aug 2016

Looks like it, says:

5link varchar(200) utf8mb4_unicode_ci

On Mon, Aug 15, 2016 at 1:22 PM, Luca Marzo notifications@github.com
wrote:

yes, i guess that it's a limit on the field.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#11612 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwPrYre8GLRGuSbs_c5uXWNo4fpJHB4ks5qgEvjgaJpZM4JkSWw
.

avatar dkanchev
dkanchev - comment - 15 Aug 2016

In that case I see two solutions:

1) Either the limit on the field should be changed in the next version of Joomla!

2) Or the UX should be changed to show an error message to the client that the link is too long.

avatar jeckodevelopment
jeckodevelopment - comment - 15 Aug 2016

I prefer the 2nd way.
A check on the field to validate input could be useful.

avatar dkanchev
dkanchev - comment - 15 Aug 2016

And if we are going to limit the user to use maximum 200 chars then one should make sure that this will work with shortened URLs.

avatar brianteeman
brianteeman - comment - 15 Aug 2016

As the maximum url length is 2038 characters in IE (longer in Firefox)
there really is no logical reasoning for this to be limited to 200 is there?

avatar dkanchev
dkanchev - comment - 15 Aug 2016

I totally agree with Brian that this is a legacy limit and it should be removed - users should not be limited if there is no reason to do this.

avatar brianteeman
brianteeman - comment - 15 Aug 2016

Note the other places where you might store a url
com_redirect varchar(2048)
com_menus link varchar(1048)

On 15 August 2016 at 12:44, Daniel Kanchev notifications@github.com wrote:

I totally agree with Brian that this is a legacy limit and it should be
removed - users should not be limited if there is no reason to do this.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#11612 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8U58SJS8cdIEEbyq_mSi7WT1yFNsks5qgFE4gaJpZM4JkSWw
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman brianteeman - change - 15 Aug 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-08-15 12:49:00
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 15 Aug 2016

Closed as we have a PR for testing see #11614


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11612.

avatar brianteeman brianteeman - close - 15 Aug 2016

Add a Comment

Login with GitHub to post a comment