Content type selection not displaying

Created on 16 September 2020, about 4 years ago
Updated 28 June 2024, 5 months ago

There is a line on the configuration page for "Choose the content type for which you want to create notification.", but there are no options displayed for published, unpublished or both available. All content always triggers an email when created. It is desirable to not email users for unpublished content in some cases.

🐛 Bug report
Status

Needs review

Version

2.8

Component

User interface

Created by

🇺🇸United States sirbanks

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • Status changed to Needs review about 1 year ago
  • 🇮🇳India shiv_yadav

    Hello sirbanks,
    I have install module .
    I showed content type. attached screenshot shot as well.

  • First commit to issue fork.
  • Status changed to Needs work 5 months ago
  • 🇮🇳India Tirupati_Singh

    Hi @ankitdebnath, I tried applying the MR!9 as patch but the patch is not being applied. Attaching screenshot for reference. Hence moving it to Needs work.

  • Assigned to PrabuEla
  • Issue was unassigned.
  • Status changed to Needs review 5 months ago
  • 🇮🇳India Tirupati_Singh

    @PrabuEla, could you provide the steps on how you have applied the patch? On applying the patch I'm getting below errors.

      - Applying patches for drupal/content_notification
        https://git.drupalcode.org/project/content_notification/-/merge_requests/9.patch (Content type selection not displaying)
    patch '-p1' --no-backup-if-mismatch -d '/app/web/modules/contrib/content_notification' < '/tmp/666946488f4ee.patch'
    patching file src/ContentNotificationService.php
    
    Hunk #1 succeeded at 342 (offset 74 lines).
    Hunk #2 succeeded at 360 (offset 74 lines).
    Hunk #3 FAILED at 318.
    1 out of 3 hunks FAILED -- saving rejects to file src/ContentNotificationService.php.rej
    patching file src/Controller/ContentNotificationController.php
    
    Hunk #1 succeeded at 11 with fuzz 1 (offset 3 lines).
    Hunk #2 FAILED at 63.
    Hunk #3 succeeded at 111 (offset 31 lines).
    1 out of 3 hunks FAILED -- saving rejects to file src/Controller/ContentNotificationController.php.rej
    
    patching file src/Form/ContentNotificationAddForm.php
    
    Hunk #1 succeeded at 97 (offset 20 lines).
    Hunk #2 succeeded at 287 (offset 20 lines).
    
    patching file src/Form/ContentNotificationEditForm.php
    
    Hunk #1 succeeded at 108 (offset 20 lines).
    Hunk #2 succeeded at 306 (offset 26 lines).
    
    patch '-p0' --no-backup-if-mismatch -d '/app/web/modules/contrib/content_notification' < '/tmp/666946488f4ee.patch'
    can't find file to patch at input line 18
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |From 7386550e0bbd6e13fc06c4d2ecb23d9562f3e7a8 Mon Sep 17 00:00:00 2001
    |From: ankitdebnath003 <ankit.debnath@innoraft.com>
    |Date: Thu, 4 Apr 2024 17:34:14 +0530
    |Subject: [PATCH] Issue #3171319: Add a configuration to check which type of
    | node status will trigger the mail.
    |
    |---
    | src/ContentNotificationService.php            | 35 +++++++++++++++----
    | .../ContentNotificationController.php         | 11 ++++--
    | src/Form/ContentNotificationAddForm.php       | 13 +++++++
    | src/Form/ContentNotificationEditForm.php      | 14 ++++++++
    | 4 files changed, 63 insertions(+), 10 deletions(-)
    |
    |diff --git a/src/ContentNotificationService.php b/src/ContentNotificationService.php
    |index 58f03c4..a74bfa7 100644
    |--- a/src/ContentNotificationService.php
    |+++ b/src/ContentNotificationService.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    3 out of 3 hunks ignored
    
    
    can't find file to patch at input line 81
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Controller/ContentNotificationController.php b/src/Controller/ContentNotificationController.php
    |index b289b48..ba59319 100644
    |--- a/src/Controller/ContentNotificationController.php
    |+++ b/src/Controller/ContentNotificationController.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    3 out of 3 hunks ignored
    
    
    can't find file to patch at input line 122
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Form/ContentNotificationAddForm.php b/src/Form/ContentNotificationAddForm.php
    |index 7510c32..95988b7 100644
    |--- a/src/Form/ContentNotificationAddForm.php
    |+++ b/src/Form/ContentNotificationAddForm.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    2 out of 2 hunks ignored
    
    
    can't find file to patch at input line 153
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Form/ContentNotificationEditForm.php b/src/Form/ContentNotificationEditForm.php
    |index 5a24e60..df2fe7f 100644
    |--- a/src/Form/ContentNotificationEditForm.php
    |+++ b/src/Form/ContentNotificationEditForm.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    2 out of 2 hunks ignored
    
    
    patch '-p2' --no-backup-if-mismatch -d '/app/web/modules/contrib/content_notification' < '/tmp/666946488f4ee.patch'
    can't find file to patch at input line 18
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |From 7386550e0bbd6e13fc06c4d2ecb23d9562f3e7a8 Mon Sep 17 00:00:00 2001
    |From: ankitdebnath003 <ankit.debnath@innoraft.com>
    |Date: Thu, 4 Apr 2024 17:34:14 +0530
    |Subject: [PATCH] Issue #3171319: Add a configuration to check which type of
    | node status will trigger the mail.
    |
    |---
    | src/ContentNotificationService.php            | 35 +++++++++++++++----
    | .../ContentNotificationController.php         | 11 ++++--
    | src/Form/ContentNotificationAddForm.php       | 13 +++++++
    | src/Form/ContentNotificationEditForm.php      | 14 ++++++++
    | 4 files changed, 63 insertions(+), 10 deletions(-)
    |
    |diff --git a/src/ContentNotificationService.php b/src/ContentNotificationService.php
    |index 58f03c4..a74bfa7 100644
    |--- a/src/ContentNotificationService.php
    |+++ b/src/ContentNotificationService.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    3 out of 3 hunks ignored
    
    
    can't find file to patch at input line 81
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Controller/ContentNotificationController.php b/src/Controller/ContentNotificationController.php
    |index b289b48..ba59319 100644
    |--- a/src/Controller/ContentNotificationController.php
    |+++ b/src/Controller/ContentNotificationController.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    3 out of 3 hunks ignored
    
    
    can't find file to patch at input line 122
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Form/ContentNotificationAddForm.php b/src/Form/ContentNotificationAddForm.php
    |index 7510c32..95988b7 100644
    |--- a/src/Form/ContentNotificationAddForm.php
    |+++ b/src/Form/ContentNotificationAddForm.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    2 out of 2 hunks ignored
    
    
    can't find file to patch at input line 153
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Form/ContentNotificationEditForm.php b/src/Form/ContentNotificationEditForm.php
    |index 5a24e60..df2fe7f 100644
    |--- a/src/Form/ContentNotificationEditForm.php
    |+++ b/src/Form/ContentNotificationEditForm.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    2 out of 2 hunks ignored
    
    
    patch '-p4' --no-backup-if-mismatch -d '/app/web/modules/contrib/content_notification' < '/tmp/666946488f4ee.patch'
    can't find file to patch at input line 18
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |From 7386550e0bbd6e13fc06c4d2ecb23d9562f3e7a8 Mon Sep 17 00:00:00 2001
    |From: ankitdebnath003 <ankit.debnath@innoraft.com>
    |Date: Thu, 4 Apr 2024 17:34:14 +0530
    |Subject: [PATCH] Issue #3171319: Add a configuration to check which type of
    | node status will trigger the mail.
    |
    |---
    | src/ContentNotificationService.php            | 35 +++++++++++++++----
    | .../ContentNotificationController.php         | 11 ++++--
    | src/Form/ContentNotificationAddForm.php       | 13 +++++++
    | src/Form/ContentNotificationEditForm.php      | 14 ++++++++
    | 4 files changed, 63 insertions(+), 10 deletions(-)
    |
    |diff --git a/src/ContentNotificationService.php b/src/ContentNotificationService.php
    |index 58f03c4..a74bfa7 100644
    |--- a/src/ContentNotificationService.php
    |+++ b/src/ContentNotificationService.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    3 out of 3 hunks ignored
    
    
    can't find file to patch at input line 81
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Controller/ContentNotificationController.php b/src/Controller/ContentNotificationController.php
    |index b289b48..ba59319 100644
    |--- a/src/Controller/ContentNotificationController.php
    |+++ b/src/Controller/ContentNotificationController.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    3 out of 3 hunks ignored
    
    
    can't find file to patch at input line 122
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Form/ContentNotificationAddForm.php b/src/Form/ContentNotificationAddForm.php
    |index 7510c32..95988b7 100644
    |--- a/src/Form/ContentNotificationAddForm.php
    |+++ b/src/Form/ContentNotificationAddForm.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    2 out of 2 hunks ignored
    
    
    can't find file to patch at input line 153
    
    Perhaps you used the wrong -p or --strip option?
    
    The text leading up to this was:
    --------------------------
    
    |diff --git a/src/Form/ContentNotificationEditForm.php b/src/Form/ContentNotificationEditForm.php
    |index 5a24e60..df2fe7f 100644
    |--- a/src/Form/ContentNotificationEditForm.php
    |+++ b/src/Form/ContentNotificationEditForm.php
    --------------------------
    
    File to patch: 
    
    Skip this patch? [y] 
    
    Skipping patch.
    
    2 out of 2 hunks ignored
    
    
       Could not apply patch! Skipping. The error was: Cannot apply patch https://git.drupalcode.org/project/content_notification/-/merge_requests/9.patch
    
  • 🇮🇳India PrabuEla chennai

    @Tirupati_Singh
    I have not applied the patch, I cloned the module switched to the latest MR branch and tried to add a fix but I found that the fix was already added. So I have shared a screenshot.

Production build 0.71.5 2024