@alan delval, I have fixed the issue and created Patch. please review it once
@geolim4, i have fixed the issue and created Patch. please review it once
@yaëlm, I have tested above patch locally, issue has fixed working well for me. moving RTBC++
@quilx, I have fixed the issue and created Patch , please review it once
I have fixed the issue and created Patch. please review it once
@rex.barkdoll, I have fixed the issue, and created MR please review it once
abhishek_gupta1 → made their first commit to this issue’s fork.
@el7cosmos, I have fixed the issue and created MR. please review it once
abhishek_gupta1 → made their first commit to this issue’s fork.
@codebymikey, I have fixed the issue and created MR. Please review it once
abhishek_gupta1 → made their first commit to this issue’s fork.
@yosriy. I have fixed the issue and created MR, Please review it once.
abhishek_gupta1 → made their first commit to this issue’s fork.
Hello @stomusic, I have applied above patch on local env, issue has fixed working well for me, Moving RTBC++
@duwid, I have tested above patch locally, issue has fixed. working well for me. moving RTBC++
alexpott → credited abhishek_gupta1 → .
@sarwan_verma,
Could you please ensure that you only pick up issues that are not already assigned to someone else? If an issue is already assigned, it's important to take note and avoid working on it. Overlooking this wastes time for both of us, as well as for others who may be working on the same issue. Remember, contribution is not just about earning credit; it's about respecting the efforts and time of everyone involved. Let's be mindful, considerate, and make the best use of our time.
@joshi.rohit100, I have tested locally, #2 MR fixed the issue. working for me.
@rraney, I Have fixed the issue and created MR. please review it once
@joseminosa, I have applied above patch locally, issue has fixed working for me. moving needs review
@dark05, I have tested above patch locally following steps to reproduce. issue has fixed, working well for me. Also I created MR easy to merge.
thanks
abhishek_gupta1 → made their first commit to this issue’s fork.
@daniel_j, i have applied above patch on local environment, and followed up mentioned steps to reproduce , Issue has fixed, working well for me. Also I created MR easy to merge. please review it once
thanks
abhishek_gupta1 → made their first commit to this issue’s fork.
I have resolved locally this error, but can't upload patch or MR in this entity_pdf issue . because that is related to views_bulk_operations module. you can make changes in Drupal\views_bulk_operations\Service\ViewsBulkOperationsActionProcessor->process() function
replace this line
if ($accessResult->isAllowed() === FALSE) {
to
// Only call isAllowed() if $accessResult is an instance of AccessResultInterface
if ($accessResult instanceof AccessResultInterface && $accessResult->isAllowed() === FALSE) {
and replace this line '@reason' => $accessResult->getReason(), to '@reason' => $reason,
I tried #2 approach, and it works. created patch. please review it once
@mile23, I have fixed the issue and created MR. please review it once
@danielspeicher, I have verified locally, above patch is working well for me. Also I created MR for that easy to merge. please review it once.
abhishek_gupta1 → made their first commit to this issue’s fork.
fixed the issue and created MR, please review it once
@josephgut, I have fixed the issue and created MR. please review it once
abhishek_gupta1 → made their first commit to this issue’s fork.
@kopeboy, I have fixed the issue and created MR. please review it once
@yannickoo, I have tested above path locally, fixed the issue and I created MR easy to merge. please review it once.
abhishek_gupta1 → made their first commit to this issue’s fork.
@trickfun, I have fixed the issue and created MR please review it once.
@collinhaines, I have tested locally #2 patch fixed the issue, working well for me. moving RTBC++
@Sophron, I have fixed the issue and created MR. please review it once.
abhishek_gupta1 → made their first commit to this issue’s fork.
@tjtj, I have fixed the issue and created MR. please review it once.
@killua99, MR14 is working for me. I have tested on my local working well for me .
@UriDrupal, I have fixed the issue and created MR. please review it once.
@sah62, I have fixed the issue and created MR. please review it once.
@clemorphy, I have fixed the issue and created MR. please review it once.
@ekapus, I have tested locally above MR it has fixed the issue, moving RTBC++
@mesharideb, I have tested locally patch #2 fixed the issue. Also i have created MR easy to merge . please review it once.
@naveenvalecha, Fixed the issue and created MR. please review it once.
Hello @UriDrupal, Could you please provide more information about the error, including the specific file and function where it is occurring? Additionally, please provide steps to reproduce the error, and specify which actions lead to this error.
Thank you.
@bajah1701, I have tested locally, above MR fixed the issue working well for me. moving RTBC++, thanks
@therobyouknow, I have added dependency of smart_date module in info file, created MR. please review it once.
abhishek_gupta1 → made their first commit to this issue’s fork.
@naidim, I have fixed the issue and created MR. please review it once.
@dimaro, I have tested your Patch locally following mentioned step to reproduce. Issue has Fixed now, its working well for me. Also I created MR easy to merge. moving RTBC++
abhishek_gupta1 → made their first commit to this issue’s fork.
@colinstillwell,. I have fixed the issue and created MR. please review it once.
@KlemenDEV, I have fixed the issue and created MR. please review it once.
@SirClickalot, I have fixed the issue and created Patch file, please review it once.
@Asheef, fixed the issue and created MR, please review it once.
fixed the issue and created MR. please review it once.
@ptmkenny, I have fixed the issue and created MR. please review it once.
@fotisp hello, I have tested locally, above Patch is working well for me. issue has fixed . Also I created MR easy to merge. please review it once.
abhishek_gupta1 → made their first commit to this issue’s fork.
@hongqing, I have fixed the issue and created MR , Please review it once.
@JonRonn, I have fixed the issue and created MR. please review it once.