andig / carddav2fb

Download CardDAV VCards and upload as phonebook to AVM FRITZ!Box
63 stars 19 forks source link

import_vcf_file #155

Closed blacksenator closed 5 years ago

blacksenator commented 5 years ago

Also allows import of local vcf files; refactoring of the download, convert and upload commands according to the use of sabre/vobject

andig commented 5 years ago

Ich meine überall wo localProvider aufgerufen wird einfach durch

$foo = new VcardFile($path)

ersetzen. Die Funktion liefert null Mehrwert und verwirrt einfach nur…

Viele Grüße, Andreas

On 12. Sep 2019, at 12:43, Black Senator notifications@github.com wrote:

@blacksenator commented on this pull request.

In src/functions.php https://github.com/andig/carddav2fb/pull/155#discussion_r323673244:

@@ -28,12 +29,19 @@ function backendProvider(array $config): Backend return $backend; }

+function localProvider($fullpath) Was meinst Du damit? Keine eigene Klasse VcardFile https://github.com/andig/carddav2fb/pull/155/files#diff-0e794b0bb149688422623d6ddaf4557dund statt dessen mit zusätzlichem Parameter im Konstruktor steuern, ob man vom Server oder lokal einlesen möchte? Also nur Backend und dafür dort getVcards() erweitern?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/andig/carddav2fb/pull/155?email_source=notifications&email_token=AABND36Z2M3ECGFUDTX663TQJIML5A5CNFSM4ISTO2EKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCEQHR7Y#discussion_r323673244, or mute the thread https://github.com/notifications/unsubscribe-auth/AABND37O2QNR24RT6SXBEY3QJIML5ANCNFSM4ISTO2EA.