Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
This module enables you to render a FlexSlider using referenced entities as your slides.
It provides a field formatter for entityreference fields, with the following settings;
- FlexSlider option set
- View mode for slides
- View mode for thumbnails
Sandbox URL: https://drupal.org/sandbox/bendik/1886168
Git: git clone http://git.drupal.org/sandbox/bendik/1886168.git flexslider_entityreference
Comment | File | Size | Author |
---|---|---|---|
#14 | 2013-03-01-02:52:31.jpg | 6.74 KB | drupik |
Comments
Comment #1
klausiProject 1: http://drupal.org/node/1606634
Project 2: http://drupal.org/node/1886184
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.
Comment #2
bendikrb CreditAttribution: bendikrb commentedComment #3
klausiWe are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)
Comment #4
alexmoreno CreditAttribution: alexmoreno commentedThanks for contributing your module.
I've tryed to clone the project to read the code and do some automated tests but it seems the project is empty.
Also, the main git branch should be pointing to a current drupal version, like drupal 7 or drupal 6. Currently it points to master.
Also, please, provide more information about your project and how to download it.
Runing paraview.sh also throws this (as klausi points):
It appears you are working in the "master" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.
And this validation issues in Drupal coding standards.
37 | ERROR | Inline comments must end in full-stops, exclamation marks, or
| | question marks
83 | ERROR | Inline comments must end in full-stops, exclamation marks, or
| | question marks
92 | ERROR | Inline comments must end in full-stops, exclamation marks, or
| | question marks
100 | ERROR | else must start on a new line
161 | ERROR | Files must end in a single new line character
Thank you.
Comment #5
bendikrb CreditAttribution: bendikrb commentedThanks for your review.
I am not working in the master branch. All my code is in 7.x-1.x. I have made this the default branch now.
The issues reported by paraview is now corrected.
Comment #6
alexmoreno CreditAttribution: alexmoreno commentedFrom my point of view the module can be ready to publish. I you could follow the advice of bendikrb, if you help with at least 3 or 4 reviews of other modules waiting, you'll help others to leave their sandboxes (like myself, I will have a module in a few days ready) and your module will be approved sooner.
A few tips that could be useful:
http://www.programadoresweb.net/drupal-sprint-london-january-2013/26-01-...
Thank you.
Comment #7
fagoI could use the module, so I reviewed it:
Inconsistent spelling for "Entityreference", this should follow the spelling of the entity reference project.
Credits should take place in README.txt not in @file blocks.
Use verbose variables names, $o says nothing. Also, instead of !== NULL use isset($var) unless not possible.
It's good to have that kind of protection.
This could use a comment why it's necessary to clone, e.g. to support rendering the same entity in another view-mode on itself?
Except of that the code looks pretty solid and should be ready. Good work!
Comment #8
bendikrb CreditAttribution: bendikrb commentedThanks again for your review!
I have corrected the issues you have pointed out, and added a README file.
Comment #9
alexmoreno CreditAttribution: alexmoreno commentedremember to help wit 3 or 4 reviews like this one :-)
Comment #10
bendikrb CreditAttribution: bendikrb commentedWill do!
It's not than i'm being ignorant, I'm just burried with work nowadays.
I'll do some reviewing as soon as I find the time for it :)
Comment #11
alexmoreno CreditAttribution: alexmoreno commentedSorry bendikrb, I didn't want to mean nothing about you or being disrespectful. It is just that It's the typical thing that a lot of projects lack of. Once you know how to do it, it is very easy and you will help others like you (or myself) to leave the sandbox.
Thank you, have a lovely weekend.
Comment #12
stevectorThe project page should link to the Flexslider module page http://drupal.org/project/flexslider and contain some mention of compatibility with versions 1 and 2 of flexslider module.
Comment #13
bendikrb CreditAttribution: bendikrb commentedProject page is now updated
Comment #14
drupik CreditAttribution: drupik commentedHi bendikrb.
flexslider_entityreference.module - line 6:
- It seems that you still not removed the tag @author from @file block mentioned by @fago.Add git clone command to your issue summary:
git clone http://git.drupal.org/sandbox/bendik/1886168.git flexslider_entityreference
flexslider_entityreference.info
- you have this:package = FlexSlider
, flexslider module have:package = Flex Slider
, on the modules page they are two separate groups.The rest looks great.
Good Luck :)
Comment #15
bendikrb CreditAttribution: bendikrb commentedThanks drupik.
I have now done the changes you have pointed out.
Comment #16
drupik CreditAttribution: drupik commentedCool.
git clone - I mean here, in issue summary (the first post) not on the project page, there currently is ;) http://drupal.org/node/1011698 - point 6
Comment #17
bendikrb CreditAttribution: bendikrb commentedRight, ok. Done! :)
Comment #18
drupik CreditAttribution: drupik commentedHi bendikrb.
Can you explain what I'm doing wrong? :)
5.3.10-1ubuntu3.5
Thumbnails rendered as: empty string
Rendered entity works well, as well as a image with flexslider.
Comment #19
bendikrb CreditAttribution: bendikrb commentedHi drupik,
do you have Flexslider working before trying my module?
You need the contrib library for it (jquery.flexslider.js), please see the instructions on the flexslider project page.
Also make sure you are using version 2 of the flexslider module.
Comment #20
drupik CreditAttribution: drupik commentedYes, it worked before, but I had a different version.
Yeah, now it works with entityreference! I has flexslider 7.x-1.0-rc3 because it is recommended release. You should add info about this to install flexslider 2.0, because it definitely does not work with 7.x-1.0 version. Also, you can add more specified dependencies to the .info file, see: http://drupal.org/node/542202#dependencies.
You can change back in the .info file
package = Flex Slider
, because since version 2 it is:package = FlexSlider
(my bad).It looks great, works even on responsive theme. I will definitely use this module in the future.
Changing the status to RTBC, but the rest is in the hands of git admin. To speed up the process, you can do 3 reviews (as I do now :) http://drupal.org/node/1410826
That's all from my side, keep the good work.
Greetings.
Comment #21
bendikrb CreditAttribution: bendikrb commentedThanks for your review drupik!
I have now made the flexslider dependency version specific and changed back the package string.
Comment #22
mitchell CreditAttribution: mitchell commentedThanks for your contribution, bendikrb!
I updated your account to let you promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and get involved!
Thanks to the dedicated reviewer as well.
Comment #23
bendikrb CreditAttribution: bendikrb commentedThanks alot guys!
I have promoted the project and added a release candidate;
https://drupal.org/project/flexslider_entityreference
Comment #24.0
(not verified) CreditAttribution: commentedAdded git clone