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

Collections #1810

Draft
wants to merge 14 commits into
base: master
Choose a base branch
from
3 changes: 2 additions & 1 deletion freesound/context_processor.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,8 @@ def context_extra(request):
'next_path': request.GET.get('next', request.get_full_path()),
'login_form': FsAuthenticationForm(),
'problems_logging_in_form': ProblemsLoggingInForm(),
'system_prefers_dark_theme': request.COOKIES.get('systemPrefersDarkTheme', 'no') == 'yes' # Determine the user's system preference for dark/light theme (for non authenticated users, always use light theme)
'system_prefers_dark_theme': request.COOKIES.get('systemPrefersDarkTheme', 'no') == 'yes', # Determine the user's system preference for dark/light theme (for non authenticated users, always use light theme)
'COLLECTIONS': settings.COLLECTIONS
})

return tvars
3 changes: 3 additions & 0 deletions freesound/settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,7 @@
'admin_reorder',
'captcha',
'adminsortable',
'fscollections'
]

# Specify custom ordering of models in Django Admin index
Expand Down Expand Up @@ -927,6 +928,8 @@
# -------------------------------------------------------------------------------
# Extra Freesound settings

COLLECTIONS = True

# Paths (depend on DATA_PATH potentially re-defined in local_settings.py)
# If new paths are added here, remember to add a line for them at general.apps.GeneralConfig. This will ensure
# directories are created if not existing
Expand Down
107 changes: 107 additions & 0 deletions freesound/static/bw-frontend/src/components/collectSound.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,107 @@
import {dismissModal, handleGenericModal} from "./modal";
import {showToast} from "./toast";
import {makePostRequest} from "../utils/postRequest";

const saveCollectionSound = (collectSoundUrl, data) => {

let formData = {};
if (data === undefined){
formData.name = "";
formData.collection = "";
formData.new_collection_name = "";
formData.use_last_collection = true;
} else {
formData = data;
}
makePostRequest(collectSoundUrl, formData, (responseText) => {
// CollectionSound saved successfully. Close model and show feedback
dismissModal(`collectSoundModal`); // TBC
try {
showToast(JSON.parse(responseText).message);
} catch (error) {
// If not logged in, the url will respond with a redirect and JSON parsing will fail
showToast("You need to be logged in before collecting sounds.")
}
}, () => {
// Unexpected errors happened while processing request: close modal and show error in toast
dismissModal(`collectSoundModal`);
showToast('Some errors occurred while collecting the sound.');
});
}


const toggleNewCollectionNameDiv = (select, newCollectionNameDiv) => {
if (select.value == '0'){
// No category is selected, show the new category name input
newCollectionNameDiv.classList.remove('display-none');
} else {
newCollectionNameDiv.classList.add('display-none');
}
}


const initCollectSoundFormModal = (soundId, collectSoundUrl) => {

// Modify the form structure to add a "Category" label inline with the select dropdown
const modalContainer = document.getElementById('collectSoundModal');
const selectElement = modalContainer.getElementsByTagName('select')[0];
const wrapper = document.createElement('div');
wrapper.style = 'display:inline-block;';
if (selectElement === undefined){
// If no select element, the modal has probably loaded for an unauthenticated user
console.log("select element is undefined")
return;
}
console.log("SELECT ELEMENT", selectElement);
selectElement.parentNode.insertBefore(wrapper, selectElement.parentNode.firstChild);
const label = document.createElement('div');
label.innerHTML = "Select a collection:"
label.classList.add('text-grey');
wrapper.appendChild(label)
wrapper.appendChild(selectElement)

const formElement = modalContainer.getElementsByTagName('form')[0];
const buttonsInModalForm = formElement.getElementsByTagName('button');
const saveButtonElement = buttonsInModalForm[buttonsInModalForm.length - 1];
const categorySelectElement = document.getElementById(`id_${ soundId.toString() }-collection`);
const newCategoryNameElement = document.getElementById(`id_${ soundId.toString() }-new_collection_name`);
console.log("CATEGORY SELECT ELEMENT: ", categorySelectElement);
console.log("NEW CATEGORY NAME ELEMENT: ", newCategoryNameElement);
toggleNewCollectionNameDiv(categorySelectElement, newCategoryNameElement);
categorySelectElement.addEventListener('change', (event) => {
toggleNewCollectionNameDiv(categorySelectElement, newCategoryNameElement);
});

// Bind action to save collectionSound in "add sound to collection button" (and prevent default form submit)
saveButtonElement.addEventListener('click', (e) => {
e.preventDefault();
const data = {};
data.collection = document.getElementById(`id_${ soundId.toString() }-collection`).value;
data.new_collection_name = document.getElementById(`id_${ soundId.toString() }-new_collection_name`).value;
saveCollectionSound(collectSoundUrl, data);
});
};

const bindCollectSoundModals = (container) => {
const collectSoundButtons = [...container.querySelectorAll('[data-toggle="collect-modal"]')];
collectSoundButtons.forEach(element => {
if (element.dataset.alreadyBinded !== undefined){
return;
}
element.dataset.alreadyBinded = true;
element.addEventListener('click', (evt) => {
evt.preventDefault();
const modalUrlSplitted = element.dataset.modalUrl.split('/');
const soundId = parseInt(modalUrlSplitted[modalUrlSplitted.length - 2], 10);
if (!evt.altKey) {
handleGenericModal(element.dataset.modalUrl, () => {
initCollectSoundFormModal(soundId, element.dataset.collectSoundUrl);
}, undefined, true, true);
} else {
saveCollectionSound(element.dataset.collectSoundUrl);
}
});
});
}

export { bindCollectSoundModals };
2 changes: 2 additions & 0 deletions freesound/static/bw-frontend/src/utils/initHelper.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ import { makeSelect } from '../components/select.js';
import { makeTextareaCharacterCounter } from '../components/textareaCharactersCounter.js';
import { bindUnsecureImageCheckListeners } from '../components/unsecureImageCheck.js';
import { initMap } from '../pages/map.js';
import { bindCollectSoundModals } from '../components/collectSound.js';


const initializeStuffInContainer = (container, bindModals, activateModals) => {
Expand Down Expand Up @@ -53,6 +54,7 @@ const initializeStuffInContainer = (container, bindModals, activateModals) => {
bindRemixGroupModals(container);
bindBookmarkSoundModals(container);
bindUserAnnotationsModal(container);
bindCollectSoundModals(container);
}

// Activate modals if needed (this should only be used the first time initializeStuffInContainer is called)
Expand Down
2 changes: 2 additions & 0 deletions freesound/urls.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@
import bookmarks.views
import follow.views
import donations.views
import fscollections.views
import utils.tagrecommendation_utilities as tagrec
from apiv2.apiv2_utils import apiv1_end_of_life_message

Expand Down Expand Up @@ -116,6 +117,7 @@
path('tickets/', include('tickets.urls')),
path('monitor/', include('monitor.urls')),
path('follow/', include('follow.urls')),
path('collections/', include('fscollections.urls')),

path('blog/', RedirectView.as_view(url='https://blog.freesound.org/'), name="blog"),

Expand Down
Empty file added fscollections/__init__.py
Empty file.
3 changes: 3 additions & 0 deletions fscollections/admin.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
from django.contrib import admin

# Register your models here.
6 changes: 6 additions & 0 deletions fscollections/apps.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
from django.apps import AppConfig


class FscollectionsConfig(AppConfig):
default_auto_field = 'django.db.models.BigAutoField'
name = 'fscollections'
124 changes: 124 additions & 0 deletions fscollections/forms.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,124 @@
#
# Freesound is (c) MUSIC TECHNOLOGY GROUP, UNIVERSITAT POMPEU FABRA
#
# Freesound is free software: you can redistribute it and/or modify
# it under the terms of the GNU Affero General Public License as
# published by the Free Software Foundation, either version 3 of the
# License, or (at your option) any later version.
#
# Freesound is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU Affero General Public License for more details.
#
# You should have received a copy of the GNU Affero General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
#
# Authors:
# See AUTHORS file.
#

from django import forms
from django.forms import ModelForm, Textarea, TextInput
from fscollections.models import Collection, CollectionSound
from utils.forms import HtmlCleaningCharField

#this class was aimed to perform similarly to BookmarkSound, however, at first the method to add a sound to a collection
#will be opening the search engine in a modal, looking for a sound in there and adding it to the actual collection page
#this can be found in edit pack -> add sounds
#class CollectSoundForm(forms.ModelForm):

class CollectionSoundForm(forms.Form):
#list of existing collections
#add sound to collection from sound page
collection = forms.ChoiceField(
label=False,
choices=[],
required=True)

new_collection_name = forms.CharField(
label = False,
help_text=None,
max_length = 128,
required = False)

use_last_collection = forms.BooleanField(widget=forms.HiddenInput(), required=False, initial=False)
user_collections = None
user_available_collections = None

NO_COLLECTION_CHOICE_VALUE = '-1'
NEW_COLLECTION_CHOICE_VALUE = '0'

def __init__(self, *args, **kwargs):
self.user_collections = kwargs.pop('user_collections', False)
self.user_saving_sound = kwargs.pop('user_saving_sound', False)
self.sound_id = kwargs.pop('sound_id', False)

if self.user_collections:
self.user_available_collections = Collection.objects.filter(id__in=self.user_collections).exclude(collectionsound__sound__id=self.sound_id)

# NOTE: as a provisional solution to avoid duplicate sounds in a collection, Collections already containing the sound are not selectable
super().__init__(*args, **kwargs)
self.fields['collection'].choices = [(self.NO_COLLECTION_CHOICE_VALUE, '--- No collection ---'),#in this case this goes to bookmarks collection (might have to be created)
(self.NEW_COLLECTION_CHOICE_VALUE, 'Create a new collection...')] + \
([(collection.id, collection.name) for collection in self.user_available_collections ]
if self.user_available_collections else[])

self.fields['new_collection_name'].widget.attrs['placeholder'] = "Fill in the name for the new collection"
self.fields['collection'].widget.attrs = {
'data-grey-items': f'{self.NO_COLLECTION_CHOICE_VALUE},{self.NEW_COLLECTION_CHOICE_VALUE}'}

def save(self, *args, **kwargs):
collection_to_use = None

if not self.cleaned_data['use_last_collection']:
if self.cleaned_data['collection'] == self.NO_COLLECTION_CHOICE_VALUE:
pass
elif self.cleaned_data['collection'] == self.NEW_COLLECTION_CHOICE_VALUE:
if self.cleaned_data['new_collection_name'] != "":
collection = \
Collection(user=self.user_saving_sound, name=self.cleaned_data['new_collection_name'])
collection.save()
collection_to_use = collection
else:
collection_to_use = Collection.objects.get(id=self.cleaned_data['collection'])
else:
try:
last_user_collection = \
Collection.objects.filter(user=self.user_saving_sound).order_by('-created')[0]
collection_to_use = last_user_collection
except IndexError:
pass
# If collection already exists, don't save it and return the existing one
collection, _ = Collection.objects.get_or_create(
name = collection_to_use.name, user=self.user_saving_sound)
return collection

def clean(self):
collection = self.cleaned_data['collection']
sound = self.sound_id
if CollectionSound.objects.filter(collection=collection,sound=sound).exists():
raise forms.ValidationError("This sound already exists in the collection")

return super().clean()

class CollectionEditForm(forms.ModelForm):

# description = HtmlCleaningCharField(widget=forms.Textarea(attrs={'cols': 80, 'rows': 10}),
# help_text=HtmlCleaningCharField.make_help_text(), required=False)

class Meta():
model = Collection
fields = ('name', 'description',)
widgets = {
'name': TextInput(),
'description': Textarea(attrs={'rows': 5, 'cols': 50})
}

def __init__(self, *args, **kwargs):
is_owner = kwargs.pop('is_owner', True)
super().__init__(*args, **kwargs)
if not is_owner:
for field in self.fields:
self.fields[field].widget.attrs['readonly'] = 'readonly'

28 changes: 28 additions & 0 deletions fscollections/migrations/0001_initial.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
# Generated by Django 3.2.23 on 2025-01-07 12:44

from django.conf import settings
from django.db import migrations, models
import django.db.models.deletion


class Migration(migrations.Migration):

initial = True

dependencies = [
migrations.swappable_dependency(settings.AUTH_USER_MODEL),
('sounds', '0052_alter_sound_type'),
]

operations = [
migrations.CreateModel(
name='Collection',
fields=[
('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
('name', models.CharField(default='', max_length=128)),
('created', models.DateTimeField(auto_now_add=True, db_index=True)),
('author', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
('sound', models.ManyToManyField(to='sounds.Sound')),
],
),
]
28 changes: 28 additions & 0 deletions fscollections/migrations/0002_auto_20250109_1035.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
# Generated by Django 3.2.23 on 2025-01-09 10:35

from django.db import migrations, models


class Migration(migrations.Migration):

dependencies = [
('sounds', '0052_alter_sound_type'),
('fscollections', '0001_initial'),
]

operations = [
migrations.RemoveField(
model_name='collection',
name='sound',
),
migrations.AddField(
model_name='collection',
name='description',
field=models.TextField(default='', max_length=500),
),
migrations.AddField(
model_name='collection',
name='sounds',
field=models.ManyToManyField(related_name='collections', to='sounds.Sound'),
),
]
Loading
Loading