sabredav_client 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 673012a844fcc49d9bdc6e0d3d26680e038a0d65
-  data.tar.gz: 3ef7b59e3f3be1d4dd979b9a11565bdda0b18476
+  metadata.gz: 4cc0b05832228bea29085253b712b72cd9c3332b
+  data.tar.gz: 98a50476b735fbdc9b664e4e8dc42de3f2ba4855
 SHA512:
-  metadata.gz: 5bde2b10dac16f4ae9115f1822a27244979e14234b9fb216a4b83b13b2fbd32877fc252c54d3ad2bffc7c03209661dd790b0dd293cb42eecf9a60e0df4bf3e6a
-  data.tar.gz: 8dda7560901ade742fb28e5b7379f42a55ec9896d9ff341ffa979d43301bd60c3038034c50e440c4a3365d6b832b0941ee5d4f9a2b7011e572fd70b90fdb01fd
+  metadata.gz: c74076b2c3c17556c50ed4f80ff5659ae18b0ff79df53153fea7c394b951ceec412cf96cab45e3da4f22e36a8d43dc330f06ad913090d32c63957bbc83eb5da1
+  data.tar.gz: 67e6f8fa5b7c4433767d2a43b95f768d45f811ef3f949022ef4f6c9ecd3f86e4d3c7e1cae4617c72587d9a97741a3bd4e16e6bd983df87a79ae5bd4af3964301

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    sabredav_client (0.1.1)
+    sabredav_client (0.1.2)
       builder (~> 3.2)
       net-http-digest_auth (~> 1.4)
 

data/lib/sabredav_client/calendar.rb

@@ -97,9 +97,11 @@ module SabredavClient
         sharee[:status] = :accepted unless REXML::XPath.first(entry, "//cs:invite-accepted").nil?
         # URI depends on a custom plugin
         begin
-          sharee[:uri]  = REXML::XPath.first(entry, "//cs:uri").text
+          sharee[:uri]        = REXML::XPath.first(entry, "//cs:uri").text
+          sharee[:principal]  = REXML::XPath.first(entry, "//cs:principal").text
         rescue
           #sharee[:uri] = "Property not supported by SabreDAV server"
+          #sharee[:uri] = "Property not supported by SabreDAV server"
         end
         sharees.push(sharee)
       end

data/lib/sabredav_client/principal.rb

@@ -10,7 +10,7 @@ module SabredavClient
     def create(email, displayname = nil)
       header  = {content_type: "text/xml", depth: "1"}
       body    = SabredavClient::XmlRequestBuilder::MkcolPrincipal.new(email, displayname).to_xml
-      req = client.create_request(:mkcol, header: header, body: body)
+      req     = client.create_request(:mkcol, header: header, body: body)
 
       res = req.run
       if res.code.to_i.between?(200,299)
@@ -20,6 +20,21 @@ module SabredavClient
       end
     end
 
+    def update(email: "", displayname: "")
+      header  = {content_type: "application/xml"}
+      body    = SabredavClient::XmlRequestBuilder::ProppatchPrincipal.new(email, displayname).to_xml
+      req     = client.create_request(:proppatch, header: header, body: body)
+
+      res     = req.run
+
+      if res.code.to_i.between?(200,299)
+        true
+      else
+        SabredavClient::Errors::errorhandling(res)
+      end
+
+    end
+
     def delete
       #FIXME seems like deleting a principal is forbidden by sabredav
       req = client.create_request(:delete)

data/lib/sabredav_client/version.rb

@@ -1,3 +1,3 @@
 module SabredavClient
-  VERSION="0.1.2"
+  VERSION="0.1.3"
 end

data/lib/sabredav_client/xml_request_builder/proppatch_principal.rb

@@ -0,0 +1,25 @@
+module SabredavClient
+  module XmlRequestBuilder
+
+    class ProppatchPrincipal < Base
+      attr_accessor :email, :displayname
+
+      def initialize(email, displayname)
+        @email       = email
+        @displayname = displayname
+        super()
+      end
+
+      def to_xml
+        xml.d :propertyupdate, SB_NAMESPACES do
+          xml.d :set do
+            xml.d :prop do
+              xml.tag! "sb:email-address", email unless email.empty?
+              xml.d :displayname, displayname unless displayname.empty?
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/sabredav_client/xml_request_builder.rb

@@ -1,6 +1,6 @@
 require 'builder'
 
 ['base.rb', 'propfind_calendar.rb', 'mkcalendar.rb', 'post_sharing.rb', 'report_vevent.rb', 'report_vtodo.rb', 'mkcol_principal.rb',
-'report_event_changes.rb', 'propfind_owner.rb', 'proppatch_owner.rb', 'propfind_invite.rb'].each do |f|
+'report_event_changes.rb', 'propfind_owner.rb', 'proppatch_owner.rb', 'propfind_invite.rb', 'proppatch_principal.rb'].each do |f|
     require File.join( File.dirname(__FILE__), 'xml_request_builder', f )
 end

data/spec/fixtures/xml_request_builder/proppatch_principal.xml

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<d:propertyupdate xmlns:d="DAV:" xmlns:sb="http://sabredav.org/ns">
+  <d:set>
+    <d:prop>
+      <sb:email-address>update@test.de</sb:email-address>
+      <d:displayname>David B.</d:displayname>
+    </d:prop>
+  </d:set>
+</d:propertyupdate>

data/spec/sabredav_client/principal_spec.rb

@@ -12,7 +12,7 @@ describe SabredavClient::Client do
     end
   end
 
-  describe "principal" do
+  describe "create" do
     email = "test@mail.de"
     description = "a random description"
 
@@ -35,4 +35,17 @@ describe SabredavClient::Client do
       }.to raise_error(SabredavClient::Errors::NotAllowedError)
     end
   end
+
+  describe "update" do
+
+    it "updates email and displayname" do
+      email       = "update@test.de"
+      displayname = "David B."
+      FakeWeb.register_uri(:proppatch, "http://user@localhost:5232/user/principals/user/", status: ["207", "OK"])
+
+      result = principal.update(email: email, displayname: displayname)
+      expect(result).to be
+
+    end
+  end
 end

data/spec/sabredav_client/xml_request_builder_specs/proppatch_principal_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+RSpec.describe SabredavClient::XmlRequestBuilder::ProppatchPrincipal do
+
+  let(:proppatch) { described_class.new(email = "update@test.de", diplayname = "David B." ) }
+
+  describe "#to_xml" do
+    let(:expected_xml) { File.read('spec/fixtures/xml_request_builder/proppatch_principal.xml') }
+
+    it "returns a valid xml" do
+      expect(proppatch.to_xml).to eq(expected_xml)
+    end
+
+    it "returns a valid xml without email address" do
+      proppatch.email = ""
+      expect(proppatch.to_xml).not_to include("sb:email-address")
+    end
+
+    it "returns a valid xml without displayname" do
+      proppatch.displayname = ""
+      expect(proppatch.to_xml).not_to include("d:displayname")
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sabredav_client
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Nicolas Schwartau
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-26 00:00:00.000000000 Z
+date: 2016-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: builder
@@ -99,6 +99,7 @@ files:
 - lib/sabredav_client/xml_request_builder/propfind_invite.rb
 - lib/sabredav_client/xml_request_builder/propfind_owner.rb
 - lib/sabredav_client/xml_request_builder/proppatch_owner.rb
+- lib/sabredav_client/xml_request_builder/proppatch_principal.rb
 - lib/sabredav_client/xml_request_builder/report_event_changes.rb
 - lib/sabredav_client/xml_request_builder/report_vevent.rb
 - lib/sabredav_client/xml_request_builder/report_vtodo.rb
@@ -116,6 +117,7 @@ files:
 - spec/fixtures/xml_request_builder/propfind_invite.xml
 - spec/fixtures/xml_request_builder/propfind_owner.xml
 - spec/fixtures/xml_request_builder/proppatch_owner.xml
+- spec/fixtures/xml_request_builder/proppatch_principal.xml
 - spec/fixtures/xml_request_builder/report_event_changes.xml
 - spec/sabredav_client/calendar_spec.rb
 - spec/sabredav_client/client_spec.rb
@@ -130,6 +132,7 @@ files:
 - spec/sabredav_client/xml_request_builder_specs/propfind_invite_spec.rb
 - spec/sabredav_client/xml_request_builder_specs/propfind_owner_spec.rb
 - spec/sabredav_client/xml_request_builder_specs/proppatch_owner_spec.rb
+- spec/sabredav_client/xml_request_builder_specs/proppatch_principal_spec.rb
 - spec/sabredav_client/xml_request_builder_specs/report_event_changes_spec.rb
 - spec/spec.opts
 - spec/spec_helper.rb