Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Sanitize tag list and title when adding/editing a source #308

Merged
merged 1 commit into from
May 4, 2013

Conversation

seanrand
Copy link
Contributor

@seanrand seanrand commented May 4, 2013

I was getting slightly annoyed with having to use trim() and strlen() all over the place, so this cleans up the tag list into a nice, trimmed list when inserting or updating a source in the database.

Snippets like these:

foreach ($tags as $tag) {
    $tag = trim($tag);
     if (strlen($tag) > 0) {
         ...

would become redundant. (Well, except for tags that have been added to the database before this change, until the source is edited again...)

SSilence added a commit that referenced this pull request May 4, 2013
Sanitize tag list and title when adding/editing a source
@SSilence SSilence merged commit e45771e into fossar:master May 4, 2013
@jtojnar
Copy link
Member

jtojnar commented Sep 27, 2020

Hi @seanrand. Previously, it was not clear if selfoss was licensed under GPL 3 only, or also any later version. Could you clarify whether you are fine with licensing your contributions here and in other PRs under GPL-3-or-later?

Thanks again and sorry for the confusion.

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

Successfully merging this pull request may close these issues.

3 participants