Merge pull request #26 from YunoHost-Apps/testing

update "master" from "testing"

.editorconfig

@@ -0,0 +1,21 @@
+# see http://editorconfig.org
+root = true
+
+[*]
+indent_style = space
+indent_size = 4
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+[*.py]
+line_length = 119
+
+[{Makefile,**.mk}]
+indent_style = tab
+insert_final_newline = false
+
+[*.yml]
+indent_style = tab
+indent_size = 4

.gitignore

@@ -1,6 +1,8 @@
 .*
 !.github
+!.editorconfig
 !.gitignore
 __pycache__
 secret.txt
 /local_test/
+/poetry.lock

Makefile

@@ -0,0 +1,37 @@
+SHELL := /bin/bash
+
+all: help
+
+help:
+	@awk 'BEGIN {FS = ":.*?## "} /^[a-zA-Z0-9 -]+:.*?## / {printf "\033[36m%-22s\033[0m %s\n", $$1, $$2}' $(MAKEFILE_LIST)
+
+check-poetry:
+	@if [[ "$(shell poetry --version 2>/dev/null)" == *"Poetry"* ]] ; \
+	then \
+		echo "Poetry found, ok." ; \
+	else \
+		echo 'Please install poetry first, with e.g.:' ; \
+		echo 'make install-poetry' ; \
+		exit 1 ; \
+	fi
+
+install-poetry:  ## install or update poetry
+	pip3 install -U poetry
+
+install: check-poetry  ## install PyInventory via poetry
+	sudo apt install python3-ldap libldap2-dev libsasl2-dev
+	poetry install
+
+update: install-poetry  ## update the sources and installation
+	poetry update
+
+local-test: check-poetry  ## Run local_test.py to run pyinventory_ynh locally
+	poetry run ./local_test.py
+
+local-diff-settings:  ## Run "manage.py diffsettings" with local test
+	poetry run python3 local_test/opt_yunohost/manage.py diffsettings
+
+
+##############################################################################
+
+.PHONY: help check-poetry install-poetry install update local-test

README.md

@@ -1,7 +1,7 @@
 # PyInventory for YunoHost
 
 [![Integration level](https://dash.yunohost.org/integration/pyinventory.svg)](https://dash.yunohost.org/appci/app/pyinventory) ![](https://ci-apps.yunohost.org/ci/badges/pyinventory.status.svg) ![](https://ci-apps.yunohost.org/ci/badges/pyinventory.maintain.svg)  
-[![Install PyInventory with YunoHost](https://install-app.yunohost.org/install-with-yunohost.png)](https://install-app.yunohost.org/?app=pyinventory)
+[![Install PyInventory with YunoHost](https://install-app.yunohost.org/install-with-yunohost.svg)](https://install-app.yunohost.org/?app=pyinventory)
 
 > *This package allows you to install PyInventory quickly and simply on a YunoHost server.
 If you don't have YunoHost, please consult [the guide](https://yunohost.org/#/install) to learn how to install it.*
@@ -87,7 +87,7 @@ drwxr-xr-x 6 pyinventory pyinventory    6 Dec  8 08:37 venv
 root@yunohost:~# cd /opt/yunohost/pyinventory/
 root@yunohost:/opt/yunohost/pyinventory# source venv/bin/activate
 (venv) root@yunohost:/opt/yunohost/pyinventory# ./manage.py check
-PyInventory v0.8.1 (Django v2.2.17)
+PyInventory v0.8.2 (Django v2.2.17)
 DJANGO_SETTINGS_MODULE='ynh_pyinventory_settings'
 PROJECT_PATH:/opt/yunohost/pyinventory/venv/lib/python3.7/site-packages
 BASE_PATH:/opt/yunohost/pyinventory
@@ -106,20 +106,18 @@ For quicker developing of PyInventory in the context of YunoHost app,
 it's possible to run the Django developer server with the settings
 and urls made for YunoHost installation.
 
-For this, just run `local_test.py` in a PyInventory virtualenv. 
-
 e.g.:
 ```bash
-~$ git clone https://github.com/jedie/PyInventory.git
 ~$ git clone https://github.com/YunoHost-Apps/pyinventory_ynh.git
-~$ cd PyInventory/
+~$ cd pyinventory_ynh/
-~/PyInventory$ make install
+~/pyinventory_ynh$ make
-~/PyInventory$ poetry shell
+install-poetry         install or update poetry
-(pyinventory-yd_5sxYx-py3.8) ~/PyInventory$ cd ../pyinventory_ynh/
+install                install PyInventory via poetry
-(pyinventory-yd_5sxYx-py3.8) ~/pyinventory_ynh$ ./local_test.py
+update                 update the sources and installation
-...
+local-test             Run local_test.py to run pyinventory_ynh locally
-Django version 2.2.17, using settings 'ynh_pyinventory_settings'
+~/pyinventory_ynh$ make install-poetry
-Starting development server at http://127.0.0.1:8000/
+~/pyinventory_ynh$ make install
+~/pyinventory_ynh$ make local-test
 ```
 
 Notes:

conf/ynh_pyinventory_settings.py

@@ -145,6 +145,15 @@ else:
 STATIC_ROOT = str(FINAL_WWW_PATH / 'static')
 MEDIA_ROOT = str(FINAL_WWW_PATH / 'media')
 
+# _____________________________________________________________________________
+# django-ckeditor
+
+CKEDITOR_BASEPATH = STATIC_URL + 'ckeditor/ckeditor/'
+
+# _____________________________________________________________________________
+# Django-dbbackup
+
+DBBACKUP_STORAGE_OPTIONS['location']=str(FINAL_HOME_PATH / 'backups')
 
 # -----------------------------------------------------------------------------
 
@@ -184,6 +193,6 @@ LOGGING = {
 # -----------------------------------------------------------------------------
 
 try:
-    from .local_settings import *  # noqa
+    from local_settings import *  # noqa
 except ImportError:
     pass

conf/ynh_urls.py

@@ -1,16 +1,25 @@
 from django.conf import settings
-from django.conf.urls import include
+from django.conf.urls import include, static
+from django.contrib import admin
 from django.urls import path
+from django.views.generic import RedirectView
 
 # settings.PATH_URL is the $YNH_APP_ARG_PATH
 if settings.PATH_URL:
+    # Prefix all urls with "PATH_URL":
     urlpatterns = [
-        # XXX: Hack - the MEDIA_URL contains the "PATH_URL" already:
+        path(f'{settings.PATH_URL}/admin/', admin.site.urls),
-        path(settings.MEDIA_URL.lstrip('/'), include('django_tools.serve_media_app.urls')),
 
-        # Prefix all urls with "PATH_URL":
+        path(f'{settings.PATH_URL}/', RedirectView.as_view(pattern_name='admin:index')),
-        path(f'{settings.PATH_URL}/', include('inventory_project.urls'))
+
+        path(f'{settings.PATH_URL}/ckeditor/', include('ckeditor_uploader.urls')),
+
+        # MEDIA_URL contains the "PATH_URL" already:
+        path(settings.MEDIA_URL.lstrip('/'), include('django_tools.serve_media_app.urls')),
     ]
+    if settings.SERVE_FILES:
+        urlpatterns += static.static(settings.STATIC_URL, document_root=settings.STATIC_ROOT)
 else:
-    # Installed to domain root, without a path prefix?
+    # Installed to domain root, without a path prefix
+    # Just use the default project urls.py
     from inventory_project.urls import urlpatterns  # noqa

local_test.py

@@ -130,6 +130,10 @@ def main():
     # conf/ynh_urls.py -> local_test/ynh_urls.py
     copy_patch(src_file=URLS_FILE, replaces=REPLACES)
 
+    with Path(FINAL_HOME_PATH / 'local_settings.py').open('w') as f:
+        f.write('# Only for local test run\n')
+        f.write('SERVE_FILES=True # used in src/inventory_project/urls.py\n')
+
     # call "local_test/manage.py" via subprocess:
     call_manage_py('check --deploy')
     call_manage_py('migrate --no-input')

manifest.json

@@ -5,7 +5,7 @@
 	"description": {
 		"en": "Web based management to catalog things including state and location etc."
 	},
-	"version": "0.8.1~ynh6",
+	"version": "0.8.2~ynh1",
 	"url": "https://github.com/jedie/PyInventory",
 	"license": "GPL-3.0",
 	"maintainer": {

pyproject.toml

@@ -0,0 +1,17 @@
+[tool.poetry]
+name = "pyinventory_ynh"
+version = "0.8.1~ynh7"
+description = "Test pyinventory_ynh via local_test.py"
+authors = ["JensDiemer <git@jensdiemer.de>"]
+license = "GPL"
+
+[tool.poetry.dependencies]
+python = ">=3.7,<4.0.0"
+pyinventory = "*"
+django-auth-ldap = "*"
+
+[tool.poetry.dev-dependencies]
+
+[build-system]
+requires = ["poetry-core>=1.0.0"]
+build-backend = "poetry.core.masonry.api"

scripts/_common.sh

@@ -28,7 +28,7 @@ pkg_dependencies="build-essential python3-dev python3-pip python3-venv git \
 	postgresql postgresql-contrib python3-ldap libldap2-dev libsasl2-dev"
 
 # PyInventory's version for PIP and settings file
-pyinventory_version="0.8.1"
+pyinventory_version="0.8.2"
 
 # Extra python packages:
 pypi_extras="django-redis django-auth-ldap"