Merge pull request #41

* Fixed `TypeError`

* Handles `EmptyEmbed` inside setter instead of set_

* Remove return and setter docstring
This commit is contained in:
Ahmad Ansori Palembani 2021-09-03 02:46:56 +07:00 committed by GitHub
parent 0f6db99c59
commit f37be7961a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -404,7 +404,10 @@ class Embed:
return EmbedProxy(getattr(self, '_image', {})) # type: ignore
@image.setter
def image(self: E, *, url: Any):
def image(self: E, url: Any):
if url is EmptyEmbed:
del self._image
else:
self._image = {
'url': str(url),
}
@ -431,9 +434,6 @@ class Embed:
The source URL for the image. Only HTTP(S) is supported.
"""
if url is EmptyEmbed:
del self.image
else:
self.image = url
return self
@ -454,16 +454,14 @@ class Embed:
return EmbedProxy(getattr(self, '_thumbnail', {})) # type: ignore
@thumbnail.setter
def thumbnail(self: E, *, url: Any):
"""Sets the thumbnail for the embed content.
"""
def thumbnail(self: E, url: Any):
if url is EmptyEmbed:
del self._thumbnail
else:
self._thumbnail = {
'url': str(url),
}
return
@thumbnail.deleter
def thumbnail(self):
try:
@ -485,9 +483,7 @@ class Embed:
url: :class:`str`
The source URL for the thumbnail. Only HTTP(S) is supported.
"""
if url is EmptyEmbed:
del self.thumbnail
else:
self.thumbnail = url
return self