collective / collective/collective.oembed

@@oembed-info view only exists for content items

Open
#9 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
6
Forks
6
PR merge metrics
No merged PRs in 30d

Description

collective/oembed/configure.zcml defines an oembed-info view for Archetypes and Dexterity content type objects:

```


```

Our Funnelback search engine is finding oembed discovery links in other places, for example on:

http://www.ucs.cam.ac.uk/++contextportlets++plone.rightcolumn/uis-jobs

I have worked around this by adding a fallback/default view for all objects:

```

```

However I think that collective/oembed/provider.py

```
class OEmbedProvider(BrowserView):
...
def add_context_info(self, ob, site):
....
info = ob.restrictedTraverse("@@oembed-info")()
```

should probably catch AttributeError exceptions.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in collective/oembed/provider.py at OEmbedProvider.add_context_info and inspect the restrictedTraverse("@@oembed-info") call. Review collective/oembed/configure.zcml to compare the registered view interfaces, then verify that discovery processing handles objects without that view as intended, including the portlet URL example.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.