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

Comment link redirects to random betting website #6538

Closed
sergiuionescu opened this issue Oct 10, 2024 · 2 comments · Fixed by #6541
Closed

Comment link redirects to random betting website #6538

sergiuionescu opened this issue Oct 10, 2024 · 2 comments · Fixed by #6541

Comments

@sergiuionescu
Copy link

Bug Report

Q A
Version 4.1.1

Summary

\Doctrine\DBAL\Schema\PostgreSQLSchemaManager::_getPortableTableIndexesList doc block contains a link to a website that now redirects to a betting website:

/**
     * {@inheritDoc}
     *
     * @link http://ezcomponents.org/docs/api/trunk/DatabaseSchema/ezcDbSchemaPgsqlReader.html
     */
    protected function _getPortableTableIndexesList(array $tableIndexes, string $tableName): array
    {

Current behaviour

The doc bloc is redundant.

How to reproduce

Open the file and see the doc bloc.

Expected behaviour

The doc block should only link to reputable websites, only if further context is required to describe the chosed solution.

@derrabus
Copy link
Member

Seems like somebody did not renew the domain for ez Components and some shady person took advantage of that. 🤦🏻‍♂️

Thank you for pointing that out to us.

@derrabus derrabus linked a pull request Oct 10, 2024 that will close this issue
derrabus pushed a commit that referenced this issue Oct 10, 2024
|      Q       |   A
|------------- | -----------
| Type         | improvement
| Fixed issues | #6538
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants