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

implement sensorloggercontroller::sharingOut #19

Open
alexstocker opened this issue Mar 18, 2017 · 7 comments
Open

implement sensorloggercontroller::sharingOut #19

alexstocker opened this issue Mar 18, 2017 · 7 comments

Comments

@alexstocker
Copy link
Owner

https://github.com/alexstocker/SensorLogger/blob/devel/controller/sensorloggercontroller.php#L137

@alexstocker
Copy link
Owner Author

@issb-gh
Was es hier auf jeden fall braucht ist eine Entscheidung ob es eine gute Idee ist oc_shares zu nutzen oder einen eigenen Table.

Sieh dir mal das oc_shares Schema an, das IManger (/lib/public/share) und IShare Interface und die konkrete Implementierung Share20OCS (/apps/file_sharing/lib/API). Ich denke so ähnlich könnte das funktionieren.

OCP\Share\IManager
OCA\Files_Sharing\API\Share20OCS

And brauchen wir ein paar konkrete implementierungen
Create SensorLoggerController::createShare
Create SensorLoggerController::deleteShare
Create SensorLoggerController::shareLink or sharePublic ???

etwas JavaScript
vermutlich ein wenig CSS
und möglicherweise SVG/Images

Ich denke wir verwenden sofern sinnvoll, core sutff.

Da ich es noch immer nicht geschafft hab die Sidebar und JS zu überarbeiten wäre das eine gute Gelegenheit damit anzufangen. Aber für die Sidebar und JS braucht es eigene Issues. Da steckt schon zu viel (Müll) drin.

Falls du Fragen zu Interfaces und speziell zu Dependency Injection (Owncloud DI Container https://doc.owncloud.org/server/10.0/developer_manual/app/fundamentals/container.html) haben solltest, bitte Frage mich bevor du dir den Kopf zerbrichts. Mein Wissen ist zwar nicht allumfassend aber etwas geht schon.

@issb-gh
Copy link
Collaborator

issb-gh commented Feb 8, 2019

Au fein, danke. Ja das dev manual habe ich schon entdeckt und bin gerade beim Studieren.
https://doc.owncloud.org/server/10.0/ownCloudDeveloperManual.pdf (ShareController)

Ok, mit der SideBar beginnen ist eine gute Idee. Ich habe die js-Script-Datei script.js bereits für den Branch devel-jb bearbeitet und dabei festgestellt, dass ich nicht den Sinn jeder Funktion verstanden habe.
Ich würde sehr gern zurückfragen, nur weiß ich momentan nicht, wo ich beginnen soll.
Das Owncloud-Developer-Tuturial wird mir sicher helfen, aber nach den Share-Funktionen suche ich noch...

...
Create SensorLoggerController::shareLink or sharePublic ???
...
Warum das Fragezeichen. Beide Funktionen sind zumindest für mich sehr wichtig.

@alexstocker
Copy link
Owner Author

Ich hätte gerne mehr scope und closure weniger anonymous. Bin aber immer wieder davon gelaufen ;-)

(function ($,SensorLogger) {

    SensorLogger = {
        $sidebar : $('#app-sidebar'),
        $buttons : $('.app-sensorlogger > .button')
    }


    SensorLogger.SideBar = function(settings)  {
        if ( settings.action === 'toggle') {
            SensorLogger.$sidebar.toggle();
        }
        if ( action === 'hide') {
            SensorLogger.sidebar.hide();
        }
        if ( action === 'show') {
            SensorLogger.sidebar.show();
        }
    }
    
    SensorLogger.Buttons = function() {
        SensorLogger.buttons.each(function(index,element){
            /* assign to te elmenet what to do on click */
            let $element = $(element);
            $element.on('click',function () {
                if($element.hasClass('.someclass'))  {
                    /* what should be done */
                }
            })
        })
    }

    SensorLogger.initialize = function() {
        /* define what to initialize */
        SensorLogger.Buttons();
        SensorLogger.SideBar();
    };

    SensorLogger.initialize();

})(jQuery,SensorLogger);

@alexstocker
Copy link
Owner Author

Create SensorLoggerController::shareLink or sharePublic ???

Ich denke am Ende bedeutet beides das selbe

@issb-gh
Copy link
Collaborator

issb-gh commented Feb 8, 2019

Create SensorLoggerController::shareLink or sharePublic ???
Ich denke am Ende bedeutet beides das selbe

Ok, programmiertechnisch vielleicht, aber sharePublic ist der öffentliche Link auch für nichtOwncloudUser und shareLink ist der Link für authentifizierte OC-User. Sehe ich das richtig?

alexstocker added a commit that referenced this issue Feb 11, 2019
@alexstocker
Copy link
Owner Author

Create SensorLoggerController::shareLink or sharePublic ???
Ich denke am Ende bedeutet beides das selbe

Ok, programmiertechnisch vielleicht, aber sharePublic ist der öffentliche Link auch für nichtOwncloudUser und shareLink ist der Link für authentifizierte OC-User. Sehe ich das richtig?

Da bin ich mir nicht sicher. Da ein shareLink mit einem Passwort versehen werden kann und damit per se nicht mehr Public ist.
Evtl. sollte sich shareLink nur auf das Versenden des mittels sharePublic erstellten Links an eine Email-Adresse beziehend. Ungeachtet der Möglichkeit eines Passwortschutzes oder Ablaufdatums.
Die Freigabe an OC User/Gruppen sollte über createShare laufen denke ich.
So oder so sollten wir uns an der Methodik/Strategie der Core Implementierung orientieren.

@alexstocker
Copy link
Owner Author

auswahl_004

Edit und Share Icon hinzugefügt. Delete Icon entfernt weil es WipeOut (löscht alle zum Device gehörenden Logs und DataTypes) schon gibt.

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

No branches or pull requests

2 participants