Skip to content
Snippets Groups Projects

Feature/loader

2 unresolved threads
Closed Krzysztof Dyszczyk requested to merge feature/loader into devel
2 unresolved threads

TO NIE JEST DO ZMERGOWANIA Raczej chodzi o to, żeby było widać jaki jest zamysł

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
52 if ( $requirements_checker->are_requirements_met() ) {
53 if ( ! class_exists( 'WPDesk_Plugin_Info' ) ) {
54 require_once dirname( __FILE__ ) . '/vendor/wpdesk/wp-basic-requirements/src/Plugin_Info.php';
55 }
56
57 $plugin_info = new WPDesk_Plugin_Info();
58 $plugin_info->set_plugin_file_name( plugin_basename( __FILE__ ) );
59 $plugin_info->set_plugin_dir( dirname( __FILE__ ) );
60 $plugin_info->set_class_name( 'WPDesk_Plugin_Template_Plugin' );
  • Wyciągnąłbym 'WPDesk_Plugin_Template_Plugin' gdzieś na górę, do zmiennej. Może w następnej linii po $plugin_release_timestamp.

    By Grzegorz Rola on 2018-04-16T15:29:11 (imported from GitLab)

  • jest w sumie wyciągnięte już do jednego pliku - właśnie po to, żeby było wygodnie to zmieniać. Te zmienne są na górze dlatego, że planujemy modyfikować je automatem. Nie jestem pewien czy wyciągnięcie tego do dwóch miejsc coś pomoże?

    By Dyszczo on 2018-04-16T16:35:34 (imported from GitLab)

  • W zasadzie to tylko tyle pomoże, że to co jest na 100% do zmodyfikowania będzie w jednym miejscu. Ale nie upieram się.

    By Grzegorz Rola on 2018-04-17T06:59:23 (imported from GitLab)

  • Please register or sign in to reply
  • 52 if ( $requirements_checker->are_requirements_met() ) {
    53 if ( ! class_exists( 'WPDesk_Plugin_Info' ) ) {
    54 require_once dirname( __FILE__ ) . '/vendor/wpdesk/wp-basic-requirements/src/Plugin_Info.php';
    55 }
    56
    57 $plugin_info = new WPDesk_Plugin_Info();
    58 $plugin_info->set_plugin_file_name( plugin_basename( __FILE__ ) );
    59 $plugin_info->set_plugin_dir( dirname( __FILE__ ) );
    60 $plugin_info->set_class_name( 'WPDesk_Plugin_Template_Plugin' );
    61 $plugin_info->set_version( $plugin_version );
    62 $plugin_info->set_product_id( 'WP Desk Plugin Template to change' );
    • Podobnie jak wyżej - do zmiennej. Tak aby rzeczy do modyfikowania w poszczególnych wtyczkach były w jednym miejscu.

      By Grzegorz Rola on 2018-04-16T15:31:14 (imported from GitLab)

    • Please register or sign in to reply
  • added 1 commit

    Compare with previous version

    By Dyszczo on 2018-04-16T17:03:48 (imported from GitLab)

  • closed

    By Dyszczo on 2018-05-23T09:34:37 (imported from GitLab)

  • Please register or sign in to reply
    Loading