🇦🇺Australia @imclean

Commerce Payment Fee → and Commerce Payment Fee Stripe → could be an option.

Adds support for filtering on empty and not empty. There may be a neater way to do this but it is functional.

This is actually possible. EmptyFilter() works as advertised, just need to work out how to negate it, presumably with a notExpression.

Or not. Google's AI states:

The Filter.Empty filter in Google Analytics Data API v1beta is not a standard filter type and doesn't exist as a defined object or class within the API's documentation. It's likely a placeholder or a reference to a filter that does not apply any filtering logic, essentially representing an unfiltered view of the data.

Looking at the code, it doesn't look like it does anything.

@megan_m I've created a new issue: ✨ Add support for EmptyFilter Active

imclean → created an issue.

I'm also having this issue when using a string filter on a custom dimension.

Other things I've tried result in #3346317-7: Views filter string not working → :

Error occurred! Google\ApiCore\ApiException: { "message": "Unsupported regular expression syntax in value. See https:\/\/github.com\/google\/re2\/wiki\/Syntax for supported syntax. invalid perl operator: (?!", "code": 3, "status": "INVALID_ARGUMENT", "details": [] }

Clearing the links and rechecking fixed the problem.

Using the MR with 2.1.x, I'm seeing the following error when trying to view a report:

Drupal\Component\Plugin\Exception\PluginNotFoundException: The "" entity type does not exist. in Drupal\Core\Entity\EntityTypeManager->getDefinition() (line 142 of /web/core/lib/Drupal/Core/Entity/EntityTypeManager.php).

The issue seems to come from here:

/web/modules/contrib/linkchecker/src/LinkCheckerLinkAccessControlHandler.php(68): Drupal\linkchecker\Entity\LinkCheckerLink->getParentEntity()
/web/modules/contrib/linkchecker/src/LinkCheckerLinkAccessControlHandler.php(58): Drupal\linkchecker\LinkCheckerLinkAccessControlHandler->checkParentEntityAccess()

Does the view need to be modified before this feature can be used?

@jonathanshaw I'm not sure what you mean, can you explain the exact scenario and how to replicate the problem?

Too late for this issue, but this looks like it could be a useful library for parsing a string of email addresses: https://packagist.org/packages/mmucklo/email-parse (github repository).

Although, when using a custom field formatter for file fields it's not clear how to set this option within the field formatter. The link URL object is generated in the preprocess function, and there's no way to pass additional attributes to the link element, only the warapper element.

imclean → created an issue.

Confirming #60 by setting the target for the link to _blank, which was my main use for this. I can't comment on anyone else's requirements.

function MY_THEME_preprocess_file_link(&$variables): void {
  $attributes = $variables['link']['#url']->getOption('attributes');
  $attributes['target'] = '_blank';
  $variables['link']['#url']->setOption('attributes', $attributes);  
}

Never mind, the error was all mine.

imclean → created an issue.

Thanks, I got around the immediate problem by moving Adminimal and Seven under custom themes.

The main issue is how to progress this issue. There can't be an official D11 release before Seven has one.

Still needs tests.

At the very least this should be updated as it's simply incorrect.

 *   - attributes: An associative array of attributes to be placed in the a tag.
 */
function template_preprocess_file_link(&$variables): void {}

file-link.html.twig:

<span{{ attributes }}>{{ link }}</span>
{% if file_size %}
  <span>({{ file_size }})</span>
{% endif %}

The attributes are applied to the wrapper element, not the a tag. See file.module for what's done with attributes.

How does this work without a base theme? Contrib Seven → doesn't have a Drupal 11 version yet 📌 Automated Drupal 11 compatibility fixes for seven Needs review , and it was removed from core in D11.

Well, I applied the patch and upgrade_status is happy.

I've added a few questions to the MR. Given the related issue is stalled, what can be done here?

That shouldn't make a difference. To start with, have a look at the information in the links I posted. This isn't an issue with the module itself.

I mean Google may require OAuth2:

• https://support.google.com/a/answer/14114704?hl=en
• https://support.google.com/accounts/answer/6010255?hl=en

Are you using your full email address as the username?

do you mean drupal/oauth2? Because it doesn't have a stable version.

Whether to use it or not is up to you, that's not something I can help you with.

That said, it looks like there is a stable release, as well as a module on Github which integrates the OAuth2 module with PHPMailer SMTP.

• https://github.com/iagsdc/oauth2_client_gmail_plugin
• https://www.drupal.org/project/oauth2_client →

Are you using Basic authentication? Your Gmail account may require OAuth2.

imclean → created an issue.

Committed to 2.3.x along with some other minor tidying. Markdown in Drupal doesn't yet have coding standards: 🌱 Adopt CommonMark spec for Markdown files Needs work

D11 is already supported, the change here has been added to the 2.3.x branch.

Not directly but you could integrate it with GeoIP API → to provide that information.

Under "Debug level" select "Full communication.

Your username is your full email address and your password is case sensitive.

There is a core issue which might be related: 🐛 Issue with HTML ` ` not being correctly filtered out from URLs Active

The problem also occurs when using a "REST Export" display and selecting "Data export" as the style.

render() in Drupal\views_data_export\Plugin\views\style\DataExport uses core's serializer so I guess ultimately the problem should/could be fixed there, but it would be useful if this module supported the option to decode HTML entities.

system_time_zones() is deprecated → .

Reopening as a feature request. It would be great to be able to filter based on numbers and dates, as per #5. NW for the 2.0.x branch.

imclean → created an issue. See original summary → .

#76:

+++ b/core/modules/file/file.module
@@ -1435,7 +1435,7 @@ function file_managed_file_save_upload($element, FormStateInterface $form_state)
-    \Drupal::logger('file')->notice('The upload directory %directory for the file field %name could not be created or is not accessible. A newly uploaded file could not be saved in this directory as a consequence, and the upload was canceled.', ['%directory' => $destination, '%name' => $element['#field_name']]);
+    \Drupal::logger('file')->notice('TThe upload directory %directory for the %name field could not be created or is not accessible, so the uploaded file was not saved.', ['%directory' => $destination, '%name' => $element['#field_name']]);

From the issue summary:

The problem being that on non-entity forms #field_name is not set

This issue is about removing the reliance on $element['#field_name'] being set.

Another way this problem shows itself is if the %name parameter is NULL when file.module logs an error, then the dblog page displays a white screen with the text "The website encountered an unexpected error. Try again later." The only way to fix this is to delete the entry in the watchdog table.

Here's a slightly different view based on a specific situation. I'm a maintainer of https://www.drupal.org/project/file_upload_options → and accepted a Tata employee as co-maintainer. I looked into this person and the company they work for and noticed all the requests to be a co-maintainer, many of which were rejected.

The module has very low usage and I haven't been maintaining it as much as I'd hoped to. As expected, so far only D11 compatibility has been addressed but I'm hoping he will show more interest in improving the module. Either way, I'm monitoring commits and issues and if it doesn't work out I'll remove him as co-maintainer.

This is something I've done in the past when people have offered to help, even ones with experience, then done absolutely nothing apart from getting their name on the maintainers list.

This is possible by creating a view. There are a number of ways but I've used a view of "products".

- Add a relationship to Product Variation
- Add a relationship to Stock Location using the above relationship
- Add required fields (Product title, Stock Location: Name, Local Stock Location Level: Quantity)

Make sure you've run cron after stock transactions as that's when the data gets added to the commerce_stock_location_level table.

Bunny.net's oembed response doesn't return thumbnail dimensions so resulted in an error when trying to add a video. After applying the changes in the patch manually the oembed resource was accepted.

One related problem is that Bunny.net returns a thumbnail URL, which means it should also return the thumbnail dimensions according to the oEmbed specs, but it doesn't.

This would be a task, see Issue Categories → .

imclean → created an issue.

See #5, #7, and #10. Reopen if you're interested enough to provide a solution where the key module is supported and optional

A link to instructions on how to integrate with the Key module has long been on the project page → .

nicxvan → credited imclean → .

imclean → created an issue.

Thanks for the offer. I've made you a maintainer as I have limited time to work on this module. Please review outstanding issues as well as there is more to do than just make it D11 compatible.

Twitchy finger accidentally created the MR, please ignore.

imclean → changed the visibility of the branch 3418296-no-supported-release to hidden.

imclean → made their first commit to this issue’s fork.

The other issue has more action and this was created 23 minutes after it.

I've started a new branch for 10.3.x and above. @sarwan_verma can you update the MR for this branch?

Thanks for the offer @manishvaity, you are now a maintainer.

In terms of content, I agree with #127 and #199. It would be good to show off the product, including some admin screens.

Unfortunately this change means that we will lose compatibility with Drupal 10.2 and below.

It might be worth creating a new major version for 10.3 and up.

I've tested and reviewed the code and it does what it claims to do. Although I alone am not the community, if you're happy for a single review then I'll update the status.

This works well and is a valuable addition to the module. We use Views Data Export for a lot of our clients and being able to export the data for processing makes this even more useful.

@qpro, just add the URLs for all the environments, the field accepts multiple values.

There are many unrelated changes in the MR, it should replicate the patch in either #19 or #27 (I haven't reviewed #27 properly though.)

Add PHPMailer SMTP.

I haven't made the exact changes in the MR, which were hard to follow, but I'm not seeing any errors.

This is in dev.

The iframe element will have width="" and height="" attributes. The width and height values returned from the oembed provider will allow you to work out the aspect ratio, but this would require more than just CSS to make use of.

That said, I tend to use FitVids → to solve this problem.

Ref: https://www.php.net/manual/en/function.realpath.php

Note:

The running script must have executable permissions on all directories in the hierarchy, otherwise realpath() will return false.

If a parent directory doesn't exist then realpath will return false, excluding it from this module's file_exists check and therefore not showing it in the "not on server report".

Never mind. Go to /admin/appearance/settings/civic_subtheme, go to "Content provisioning" and click "Provision content".

Media Download Tracker → is an alternative I've written for tracking downloads not using entity_usage.

I'm seeing this error every no matter how I install the theme and subtheme so I'm not sure my "steps to reproduce" would actually help.

I've tried following:

- the project page: https://www.drupal.org/project/civictheme →
- the docs: https://docs.civictheme.io/development/drupal-theme, https://docs.civictheme.io/development/drupal-theme/sub-theme
- the "Next steps" after installing civic theme subtheme dependencies:

    npm install
    npm run build
    npm run storybook

  Enable theme in Drupal:

    drush theme:enable civictheme -y
    drush config-set system.theme default civictheme
    drush theme:enable dink_civic -y
    drush config-set system.theme default dink_civic

I've tried via GUI and Drush for each step. Using Drush to install CivicTheme produced the following "warnings":

drush then civictheme
 [warning] The "extra_field_block:node:civictheme_page:content_moderation_control" was not found
 [warning] The "extra_field_block:node:civictheme_page:content_moderation_control" was not found
 [warning] The "extra_field_block:node:civictheme_event:content_moderation_control" was not found
 [warning] The "extra_field_block:node:civictheme_event:content_moderation_control" was not found
 [warning] The "block_content:50c52459-c100-4e4d-92a4-9c4c77d8f8d0" was not found
 [warning] The "block_content:7cf65f98-cbae-4e9d-b6ea-7d6d1c437c1c" was not found
 [warning] The "block_content:d7098a8c-3ba3-48f7-bc0c-5787ebaa0427" was not found
 [warning] The "block_content:5c0ad15f-3e9c-4eeb-8525-7a064b06fc58" was not found
 [warning] The "block_content:b7f36176-620f-4178-aadd-9b448c610986" was not found
 [warning] The "block_content:286ff750-a9a9-423a-8a4e-515f79fc0a8f" was not found
 [warning] The "block_content:5508dc51-1cf8-4577-9038-b572fbd698ae" was not found

I came across this recently where installing the subtheme via the GUI timed out the first time then didn't seem to properly install and I got the error in 🐛 Installing civictheme via drush results in "This block is broken or missing." on main page Postponed: needs info . I uninstalled the subtheme and reinstalled it but still had the same error.

When I then set CivicTheme as the default it also had the missing block error, even though it was fine the first time I tested it. I then uninstalled the subtheme and CivicTheme and tried to reinstall CivicTheme and got the error: "Configuration (...) provided by civictheme already exist in active configuration".

This is with a fresh install of GovCMS so I can just reinstall it again but this may not be the case in all situations.

@sarwan_verma this module supports Drupal versions from 8.8 onwards. At the very least I'd like to support 9 and 10 as well as 11. This requires using the DeprecationHelper to maintain backward compatibility. This is described in the change record I linked to in #6.

I suspect the culprit might be this line in Drupal\auditfiles\Auditor\AuditFilesNotOnServer:

$target = $this->fileSystem->realpath($result->uri);

This is converting a URI found in the file_managed table to a realpath, but it appears to return false if the file parent directory doesn't exist when it needs to return the full path to the missing file.

For context, here's the entire method:

  public function getReferences(): \Generator {
    $maximumRecords = $this->auditFilesConfig->getReportOptionsMaximumRecords();
    $query = $this->connection->select('file_managed', 'fm');
    $query
      ->orderBy('changed', 'DESC')
      ->range(0, $maximumRecords)
      ->fields('fm', ['fid', 'uri']);
    /** @var array<object{uri: string, fid: string}> $results */
    $results = $query->execute()->fetchAll();
    foreach ($results as $result) {
      $target = $this->fileSystem->realpath($result->uri);
      if ($target !== FALSE && !file_exists($target)) {
        yield FileEntityReference::create((int) $result->fid);
      }
    }
  }

imclean → created an issue.

<img src="/path/to/image.png"> will also work.

@joseph.olstad phpmailer_smtp uses the PHPMailer library as much as possible rather than duplicating its functions. For example, you won't find a single preg_match in the module either, and all address parsing is done by the library.

To understand how the module works it helps to have some knowledge of the PHPMailer library.

Specifically, in the format() method of Drupal\phpmailer_smtp\Plugin\Mail\PhpMailerSmtp there is the following line:

$this->msgHTML($rendered, DRUPAL_ROOT, TRUE);

This sends the message HTML to the PHPMailer library for processing, including inline attachments. So rather than a bunch of if/then/else statement and calling the inline attachment functions directly, this leaves it up to PHPMailer to determine, including generating a cid.

Have a look at /vendor/phpmailer/phpmailer/src/PHPMailer.php for more information.

I've changed this to a task so if the documentation is lacking in this area I'm happy to accept an MR.