s9e Media Sites

s9e Media Sites 2.18.6

No permission to download
Maybe because the regex field in the BBCode setup needs to be a URL and not an iframe?

It can be anything, or rather it's a regular expression that looks like anything you want. Make sure you check the "PCRE" box in the advanced options and if it doesn't work, post your attempt in the general support forum. Judging by your post you should be able to manually create a custom site for that kind of maps.
 
I'm also thinking XF\BbCode\ProcessorAction\AnalyzeUsage::analyzeUnfurlUsage should be written as a tag hook, not a final hook as this gives access to the tag attributes which allows vastly saner parsing to support multiple tag options.
 
I'm going to look into it asap, which should be in the upcoming days/before the end of the week.

When you say "signature limits", do you mean when the whole MEDIA tag is disabled?
 
I'd like to report a bug: the IMDB links always use the translated German titles when embedded. Can I solve this somehow?

vLcBzOS.png


Case in point: https://forums.tms.sx/threads/movies-general-discussion.24954/page-18#post-336602
 
It's likely the server is based in Germany. Title is fetched from the server's ip. IMDb probably delivers German content to German IPs.

Fix would be to use xenforo proxy from another country.

I face this problem with Netflix links. My server is in Singapore. Content not available in Singapore results in failed content fetch.
 
I'd like to report a bug: the IMDB links always use the translated German titles when embedded. Can I solve this somehow?
Yes. As Chromaniac said, IMDb probably gives you a German title because your server's IP is associated with a location in Germany. The way to fix it is to ask the IMDb server to return content in English using a Accept-language: en-us, en header. However, the box you see when posting links is not from this add-on, this is a default XenForo feature called "unfurl." Armed with this information, your best course of action is to head to the feature request forum and ask for a way to set a Accept-language header when unfurl'ing, if there isn't already an option for that. In our age of distributed, geographically-diverse cloud servers, this should be useful.
 
Last edited:
Hi Joshy, I just noticed that the Apple player is not showing in Chrome. I'm using the latest update.

IN CHROME:

34adaa297bb17293be1d0c2bc43d2a03.webp

IN FIREFOX:

afc7d0ae176d8bd87a79176379d0bab6.webp

Here is the post:

Could you check to see if this can be fixed?

Thanks!

Andre
 
Could you check to see if this can be fixed?
Unfortunately that's on Apple's side. There's an uncaught DOMException caused by Apple's code that interrupts the player's loading. Basically, if your browser denies access to session storage, an error occurs and it stops loading. Session storage is usually tied to cookie preferences in browsers, so if you allow third-party cookies it should load properly. The player works fine for me in Chrome on a new profile with the default preferences, but fails to load in Incognito mode because it blocks third-party cookies.

Edit: I've just realized that embed isn't from this add-on either. The add-on only supports Apple Podcasts, this one is Apple Music from XenForo's default media sites. It doesn't make a difference here though.
 
@JoshyPHP I've encounter something of a bug with the "Media embeds BB code" option, when using "Use a custom BB code with a media attribute". It also doesn't respect max media counts or signature limits.
Sorry for not handling it sooner. I looked into it and ended up with a slightly different fix than yours but I think it should effectively work the same. The overloaded URL tags are always active and the add-on's option only affects the way URLs are converted, that's why those tags are always counted as MEDIA tags. I could add a secondary option to toggle that behaviour, but I'd like to keep the number of options to a minimum unless there's a practical use case for it. As long as the overloaded tags are identified as media embeds, there shouldn't be a need for it but of course I'll reconsider if I'm wrong.

I don't have an automated way to test that kind of issues so I wrote down the steps to reproduce on GitHub in issues #3 and #4. I'm attaching an update for the add-on that contains those changes, let me know how it works for you.

On a tangentially related note, PHP 7.3 is EOL and about to be removed from my distribution's package manager. It's become harder for me to run XenForo 2.0 and future versions of the add-on may be solely tested on XenForo 2.2. If anybody runs XenForo 2.1 or earlier, please let me know.
 

Attachments

Sorry for not handling it sooner. I looked into it and ended up with a slightly different fix than yours but I think it should effectively work the same. The overloaded URL tags are always active and the add-on's option only affects the way URLs are converted, that's why those tags are always counted as MEDIA tags. I could add a secondary option to toggle that behaviour, but I'd like to keep the number of options to a minimum unless there's a practical use case for it. As long as the overloaded tags are identified as media embeds, there shouldn't be a need for it but of course I'll reconsider if I'm wrong.
That looks like a better approach I ended up implementing, which was done fairly quickly.
 
JoshyPHP updated s9e Media Sites with a new update entry:

Updated [URL media=...] tags, updated lazy loader

Overloaded URL tags in the form [URL media=...] are now processed as and counted as [MEDIA] tags and should be subject to the same restrictions. Thanks to Xon for the helpful report.

JavaScript event listeners used for lazy loading are now passive and apply during the capture phase. This isn't expected to make a practical difference but it may help with inconsiderate third-party scripts that hijack events.

Note that support for XenForo 2.0 and 2.1 has become...

Read the rest of this update entry...
 
Tapatalk don't work with TikTok Videos, is it possible to make TikTok-Videos to work also in Tapatalk?
Or is ist a Tapatalk Problem?
KR
 
It's a Tapatalk thing. As I recall, they only support a handful media sites in their app and TikTok doesn't seem to be one of them.
 
JoshyPHP updated s9e Media Sites with a new update entry:

Lazy loader improvements

A number of improvements have been made to the lazy loader. The goal is to generally improve efficiency and make dynamically-sized embeds such as tweets as nondisruptive as possible.
  • The size of dynamically-sized embeds (e.g. tweets) is cached locally in the browser. This will improve the user experience on repeat viewing or when the same embed is shown multiple times on a page.

  • Visible embeds are loaded as soon as possible. Previously, the lazy loader waited a tiny fraction...

Read the rest of this update entry...
 
  • Like
Reactions: Xon
I installed the latest version and just noticed a rare case with a post with several YouTube videos appear to auto-expand. May be a conflict with 8WayRun XenPorta 2 add-on which adds a 'click to expand' for longer article posts but I don't recall seeing it before.

Example page 2 - 4 videos, cuts off.
Example page 2 - 2 videos, doesn't cut off.
 
May be a conflict with 8WayRun XenPorta 2 add-on which adds a 'click to expand' for longer article posts but I don't recall seeing it before.
You're right, looks like whatever computes the height of the quote has some issues with those embeds. For reference, there's a template modification s9e_MediaSites_EmbedsInQuotes that shrinks default-sized embeds such as YouTube down to 270 × 151 inside of expandable quotes.
CSS:
.is-expandable:not(.is-expanded) [data-s9e-mediaembed]
{
	width: 270px;
	transition: none;
}

I don't know whether the thing that handles expandable quotes is from XenForo or custom from XenPorta. Post in the XenPorta support thread and somebody should be able to tell you. Tag me and I'll subscribe to it, in case there's anything I can do to help.
 
Last edited:
Hi, first of all, thank you for this fantastic addon. There is just one problem: In the EU it is required by law that a user consents before his data is sent to third parties. This is normally done by a checkbox like in the attached screenshot. It says: External Content. The small text below reads "I agree that external content is shown to me. That means personal data may be transmitted to third parties. More information can be found in our privacy policy". This would be a very important feature. Is it possible to implement this ?
 

Attachments

  • consent.webp
    consent.webp
    20.6 KB · Views: 9
There is just one problem: In the EU it is required by law that a user consents before his data is sent to third parties.
:rolleyes:


"Contrary to popular belief, the EU GDPR (General Data Protection Regulation) does not require businesses to obtain consent from people before using their personal information for business purposes."
 
Back
Top Bottom