Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Entity Ref Bootstrap Accordion / Tab Formatter:
Simple Bootstrap Accordion / Tab Field formatter for field types "entity_reference , entity_reference_revisions".
So it will be available Paragraphs module as well.
No library used except Bootstrap.
https://www.drupal.org/project/entity_bs_accordion_tab_formatter
git clone --branch 8.x-1.x https://git.drupal.org/project/entity_bs_accordion_tab_formatter.git
Comments
Comment #2
PA robot CreditAttribution: PA robot commentedProject 1: https://www.drupal.org/node/2902793
Project 2: https://www.drupal.org/node/2902792
As successful completion of the project application process results in the applicant being granted the 'Create Full Projects' permission, there is no need to take multiple applications through the process. Once the first application has been successfully approved, then the applicant can promote other projects without review. Because of this, posting multiple applications is not necessary, and results in additional workload for reviewers ... which in turn results in longer wait times for everyone in the queue. With this in mind, your secondary applications have been marked as 'closed(duplicate)', with only one application left open (chosen at random).
If you prefer that we proceed through this review process with a different application than the one which was left open, then feel free to close the 'open' application as a duplicate, and re-open one of the project applications which had been closed.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #3
PA robot CreditAttribution: PA robot commentedGit clone command for the sandbox is missing in the issue summary, please add it.
We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #4
netsliverComment #5
netsliverComment #6
netsliverComment #7
PA robot CreditAttribution: PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpsgitdrupalorgprojectentity_bs_accordion_...
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #8
netsliverComment #9
netsliverComment #10
PA robot CreditAttribution: PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #11
netsliverComment #12
nehajyoti CreditAttribution: nehajyoti as a volunteer and at QED42 commentedHi,
There are numerous Code sniffer and Indentation Issues:
Please fix - https://pareview.sh/node/2616
Comment #13
nehajyoti CreditAttribution: nehajyoti as a volunteer and at QED42 commentedHi,
There are numerous Code sniffer and Indentation Issues:
Please fix - https://pareview.sh/node/2616
Comment #14
PA robot CreditAttribution: PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #15
netsliverComment #16
netsliverNew project Webform Simplenews Handler
Comment #17
apadernoThank you for your contribution!
We review a single project and a single branch of that project. The purpose of the application is changing the roles assigned to the user who created the project, not the status of the project.
Please choose the branch you want reviewed and edit the application to reflect your choice.
Comment #18
netsliverComment #19
netsliverComment #20
netsliverComment #21
netsliverComment #22
apadernoComment #23
apadernoThank you for your contribution!
We just accept one application per user. Any application opened after the first one is closed settings its status to Closed (duplicate).
The purpose of these applications is not reviewing every project, but give users the vetted role, which allows them to change the status of their projects (the ones they already created or the ones they will create).
Comment #26
apadernoComment #27
vuilThank you for the contribution!
Please fix the following issues:
1. Please update your
js/bootstrap-responsive-tabs/README.md
file - each row can not be exceeds 80 chars per line.More info: Drupal Coding Standard
2. ESLint has been found some issues related to Javascript coding standard:
3. Please remove
implements ContainerFactoryPluginInterface
because it exists in the extendedFormatterBase
.The class declaration above is in
Drupal\entity_bs_accordion_tab_formatter\Plugin\Field\FieldFormatter\EntityBSAccordionTabFormatter
.4. Please add the two parameters (
$entity_type_id
,$bundle
) in the PHPDoc (annotation) because they are missing:5. You can simply use (in
public function settingsSummary()
):instead of:
until you decide the future solution.
6. It is better to handle all possible exceptions which will be raised (eventually) in method
viewElements
.7. Please don't use deprecated classes / interfaces as
Drupal\Core\Entity\EntityManagerInterface
which is deprecated in Drupal 8.0 and is removed from Drupal 9.0.Please see https://www.drupal.org/node/2549139
8. Please replace your parameter $plugin_definitions to be a parameter:
\Drupal\Core\Field\FieldDefinitionInterface $field_definition
instead of
FieldConfig $field_definition
.All points from 3. to 8. are in plugin class
EntityBSAccordionTabFormatter
.Comment #28
ankushgautam76@gmail.comFix below issue also :
ankushgautam@GGN-199732-C02ZT1F1MD6V contrib % drupal-check -ad entity_bs_accordion_tab_formatter
4/4 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
------ -------------------------------------------
Line entity_bs_accordion_tab_formatter.install
------ -------------------------------------------
17 Function libraries_get_path not found.
------ -------------------------------------------
------ --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Line src/Plugin/Field/FieldFormatter/EntityBSAccordionTabFormatter.php
------ --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
76 Unsafe usage of new static().
💡 Consider making the class or the constructor final.
179 Method Drupal\entity_bs_accordion_tab_formatter\Plugin\Field\FieldFormatter\EntityBSAccordionTabFormatter::viewElements() should return array but return statement is missing.
201 Call to an undefined method Drupal\Core\Entity\EntityInterface::get().
207 Call to an undefined method Drupal\Core\Entity\EntityInterface::get().
212 Call to an undefined method Drupal\Core\Entity\EntityInterface::get().
214 Variable $style might not be defined.
224 Variable $style might not be defined.
225 Variable $style might not be defined.
237 Variable $style might not be defined.
238 Variable $style might not be defined.
244 Variable $style might not be defined.
254 Variable $style might not be defined.
256 Variable $component_id might not be defined.
257 Variable $style might not be defined.
260 Variable $style might not be defined.
265 Variable $style might not be defined.
266 Variable $component_id might not be defined.
269 Variable $style might not be defined.
270 Variable $style might not be defined.
273 Variable $style might not be defined.
275 Variable $style might not be defined.
281 Variable $style might not be defined.
291 Variable $style might not be defined.
301 Variable $delta might not be defined.
305 Variable $component_id might not be defined.
308 Variable $style might not be defined.
309 Variable $delta might not be defined.
310 Variable $delta might not be defined.
------ --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
[ERROR] Found 29 errors
Comment #29
netsliverEverything is ok except Call to an undefined method Drupal\Core\Entity\EntityInterface::get() which I don't understand because the code works.
Comment #30
shaktikFix below issue :
Comment #31
netsliverOK it's fixed!
Comment #32
shaktikHi @netsliver,
Tested no error found.
4/4 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
[OK] No errors
Comment #33
apadernoComment #34
apadernoRemember to change status, when the project files have been changes as per reviews. Reviewers don't look at applications whose status is Needs work.
Comment #35
klausiconfig schema is missing, see https://www.drupal.org/docs/8/api/configuration-api/configuration-schema...
Otherwise looks good to me.
Thanks for your contribution, Sylvain!
I updated your account so you can opt into security advisory coverage now.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on Slack or IRC in #drupal-contribute. So, come hang out and stay involved!
Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Thanks to the dedicated reviewer(s) as well.