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

hot fix #2

Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open

hot fix #2

wants to merge 33 commits into from

Conversation

megabogus
Copy link
Contributor

No description provided.

Copy link
Contributor Author

@megabogus megabogus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hot fix

@Burus
Copy link
Member

Burus commented Mar 29, 2017

@megabogus а что ты хочешь в ветке мастер получить в итоге? зачем тебе эти правки в ней, не будет ли проблем с другими проектами? Где CHANGELOG ?

@megabogus
Copy link
Contributor Author

Сейчас есть жесткая ошибка не позволяющая сохранить лид, сейчас ветка мастер не работает в плане отправки данных формы

@megabogus
Copy link
Contributor Author

Exception Type: AttributeError at /feedback/new/
Exception Value: 'FeedbackPattern' object has no attribute 'save_in_api'

Copy link
Member

@Burus Burus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно до конца доделать эту задачу с выключением медиацентра. ДОбавить документацию с примером интеграции на уровне приложения, и удалять из кода месте, а не комментировать или по возможности и если это идеологически верно делать места конфигурируемыми.

Например на уровне шаблонов можно делать так сразу, на уровне методов с сервисами тоже, а вот на уровне наследования класса от медиацентра не уверен что сработает конфигурацию и тут нужно писать документ с примером.

return api.register['media'][self].get_instances(
consumer=self, is_public=True)
return []
#return api.register['media'][self].get_instances(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Этот код я не буду принимать. Тут надо делать универсальную конструкцию с проверкой модуля spicy.mediacenter в приложении и если его нет то метод ничего не возвразает, а вы просто комментарии ставите. Так не пойдет.


def has_attachments(self):
return bool(api.register['media'][self].get_instances(
consumer=self, is_public=True).count())
return False
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

То же самое, халтура какая-то

@@ -2,11 +2,10 @@
from django.db import models
from django.utils.translation import ugettext_lazy as _
from spicy.core.service import models as service_models
from spicy.mediacenter.abs import FatMediaConsumerModel
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут сложный момент с подклчение этого модуля. вот импорт мы уже условно проверить не сможем в зависимоти от конфигурации и тогда получается, что подключение загрузки файлов можно сделать только на уровне приложения. В тако м случае это должно быть указанно в документации модуля лдя программиста и дан пример конфигурации, а тут в коде просто удалены все эти места с комемнтариями. А сейчас получается что вы закомментили куски, отключили а как включить никому не сказали. Хотите чтобы к-то то раскомментировал код?

Делайте нормально до конца. Этот код я не приму.

@@ -6,6 +6,6 @@
{% for backend in backends %}
<li{% if tab == backend.1 %} class="active"{% endif %}><a href="{% url 'feedback:admin:edit-pattern' form.instance.pk backend.1 %}"><i class="icon-file-alt"></i> <span>{{ backend.0 }}</span></a></li>
{% endfor %}
<li{% if tab == 'media' %} class="active"{% endif %}><a href="{% url 'feedback:admin:edit-pattern-media' form.instance.pk %}" ><i class="icon-camera"></i> <span>{% trans "Attachments" %}</span></a></li>
{#<li{% if tab == 'media' %} class="active"{% endif %}><a href="{% url 'feedback:admin:edit-pattern-media' form.instance.pk %}" ><i class="icon-camera"></i> <span>{% trans "Attachments" %}</span></a></li>#}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем тут опять коментарий?

@@ -15,8 +15,8 @@
url(
r'^patterns/(?P<pattern_id>\d+)/extra/(?P<backend_name>\w+)/$',
'edit_pattern', name='edit-pattern'),
url(r'^patterns/(?P<pattern_id>\d+)/attach/$', 'pattern_media',
name='edit-pattern-media'),
#url(r'^patterns/(?P<pattern_id>\d+)/attach/$', 'pattern_media',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

жесть

url(
r'^patterns/attach/(?P<pattern_id>\d+)/$', 'pattern_media',
name='edit-pattern-media'),
#url(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Жесть

@@ -45,8 +45,6 @@ def new_feedback(request):
feedback.processing_status = defaults.SPAM
feedback.save()

resp = pattern.save_in_api(request)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А что это было?

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

Successfully merging this pull request may close these issues.

None yet

3 participants