3 maximum links in a description

[davidso...@gmail.com]

Hello Calagatorers,

I'm wondering if the "3 Maximum Links in a Description" is a new restriction? Also, has anybody had any issues with this feature? I'm attempting to update an event, and I repeatedly get this error, even when I edit the description to have less than 3 links. I'll file a proper issue after I determine that it's not just me.

Thanks!

[Amy K. Farrell]

Hi, this is a very new restriction, so please do file an issue. It
sounds as if you might have found a bug.

If I were to pick up this issue (who knows, it might happen!), it would
be helpful to me to have the text of the description you were trying to
submit (both the first time and after editing) included in the issue.

The link limit is supposed to be a spam-prevention measure. Since you're
obviously not a robot and you hit it, it's possible three is too low a
limit.

Thanks!

- Amy

[Kirsten Comandich]

It looks like it is a bug. The code is checking @event.description which is the saved description, rather than params[:description], the new description, so a description with more than 3 links is allowed to be saved, and then never allowed to be updated.

My computer isn't set up to run the specs properly right now, so I don't have a fix for the code at the moment, but I figured this would help someone else make an update.

Thanks,
Kirsten

[Matt Youell]

It's new. Looks like the feature merged yesterday. I know you've tried with fewer than 3 links in the description, but is there any chance "http" appears more than three times total? Perhaps by itself, or unrelated to a link?

--
You received this message because you are subscribed to the Google Groups "PDX Tech Calendar" group.
To unsubscribe from this group and stop receiving emails from it, send an email to pdx-tech-calen...@googlegroups.com.
To post to this group, send email to pdx-tech...@googlegroups.com.
Visit this group at http://groups.google.com/group/pdx-tech-calendar?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
 
 

[comm...@privly.org]

Ok, you all fixed this before I had time to file a bug. It works now. Thanks for all of your hard work with Calagator! :) 

[Reid Beels]

Yep, I poked at it a bit that evening and ended up finding it to be a quick fix.

Kirsten — you were spot on about checking the saved object rather than the submitted parameters. :)

Thanks for Calagatoring!

Reid

--