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

[4.1] Media webservice #35788

Merged
merged 76 commits into from
Nov 23, 2021
Merged

Conversation

laoneo
Copy link
Member

@laoneo laoneo commented Oct 8, 2021

Pull Request for Issue #34224.

Summary of Changes

This is a followup pr of #34314, where I continue the great work of @pjdevries. I had to change the endpoints slightly as adapters are now also supported. The new endpoints are:

GET {{base_path}}/api/index.php/v1/media/adapters
GET {{base_path}}/api/index.php/v1/media/adapters/{id}
GET {{base_path}}/api/index.php/v1/media/files/{path}
POST {{base_path}}/api/index.php/v1/media/files
PATCH {{base_path}}/api/index.php/v1/media/files/{path}
DELETE {{base_path}}/api/index.php/v1/media/files/{path} 

This is in early stage as system tests are still under development which will reveal some more issues along the way.

Testing Instructions

Check the system tests. If you want to test it by yourself, see #34314 for testing instructions with curl (don't forget to adapt the endpoints to the new ones mentioned above). Or use postman.

brianteeman and others added 30 commits May 30, 2021 15:45
* [4.0] Changing title to tooltip fpr template preview
* [4.0] mod_popular with disabled hits

When hits are disabled for articles it makes no sense to display the Most Read Articles module in the site or admin as the contents will never be updated.

This PR changes the output of the module so that instead of a list of non-updating articles a message is displayed.
* Use MVCFactory to create models

* CS
@alikon
Copy link
Contributor

alikon commented Nov 8, 2021

@laoneo is there some plugin for testing adapters ?

@alikon
Copy link
Contributor

alikon commented Nov 8, 2021

i've tested successfully with adapter local-images all' GET, POST,DELETE and PATCH

@laoneo
Copy link
Member Author

laoneo commented Nov 8, 2021

@laoneo is there some plugin for testing adapters ?

You can use the free version of DPMedia. It contains an FTP Adapter.

@laoneo
Copy link
Member Author

laoneo commented Nov 8, 2021

Here is the link to the latest version of DPMedia https://joomla.digital-peak.com/download/dpmedia where you will find the latest version.

@alikon
Copy link
Contributor

alikon commented Nov 8, 2021

I have tested this item ✅ successfully on 4b9245e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35788.

@alikon alikon added this to the Joomla 4.1 milestone Nov 8, 2021
Comment on lines 51 to 52
'name' => 'path',
'type' => 'STRING'

This comment was marked as abuse.

@bembelimen
Copy link
Contributor

@laoneo could you please check the conflicts, so it's ready for another test?

@Krshivam25
Copy link
Contributor

I have tested this item ✅ successfully on 4b9245e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35788.

@laoneo
Copy link
Member Author

laoneo commented Nov 23, 2021

@bembelimen synced with 4.1 and changes since the test from @alikon were only CS stuff.

@bembelimen bembelimen merged commit 379895c into joomla:4.1-dev Nov 23, 2021
@bembelimen
Copy link
Contributor

Thanks!

@laoneo laoneo deleted the j4/media/webservice branch November 23, 2021 19:21
This was referenced Nov 23, 2021
@sportegioco
Copy link

I tried to use these media services in different sites, both through curls and directly from postman but in no case work.
Other web services work regularly.

The error it displays is always the same:
"errors":["title":"Resource not found","code":404?]

Is it possible to have a little guide step by step to understand how to use them?

Thank you

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

Successfully merging this pull request may close these issues.